New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Restructure guide to focus on 5 ways to use C++ on web #91

Merged

sverhoeven merged 46 commits into master from develop

Jun 18, 2020

Member

sverhoeven commented Jun 18, 2020

Fixes #51
Fixes #66
Fixes #80
Fixes #81
Fixes #82
Fixes #83
Fixes #84
Fixes #88

jspaaks and others added 30 commits

June 10, 2020 08:42


          reflow the text for easier diffing; and moved some sections around

3a18a5e


          textual edits

cee2916


          updated toc

8d73669


          leftover text snippets are now hidden

4512d33


          Update README.md

4e54a3f


          function -> equation

18581a1


          Added first iteration of JavaScript web service chapter

a8c750b

Refs #83


          CI for javascript web service

14b7445


          Tabs not spaces

5d5363b


          Typo

1a8b6a2


          Merge pull request #86 from NLESC-JCER/66-readme-refactor-attempt-2

5a9d7da

readme refactor


          Merge remote-tracking branch 'origin/develop' into 83-js-web-service

f646ad3


          Standardize headings + use openapi instead of swagger

feb9694


          Enable emoij

a04bc44


          Re-use JSON schema snippets + misc edits


          Added OpenAPI web service using fastify-oas chapter

6786b57


          Typo

1697cd2


          Install fastify-oas in CI

71fb87a


          Capture web assembly logs in artifact

7d8dd88


          Added Long running task with worker threads chapter

759333a


          Updated changelog

ec2b192


          Add files for JavaScript threaded web service

a5c2164


          Proof read Accessing C++ function with Emscripten chapter

bf6daba


          Proofread Long running task with worker threads

a6b246b


          Test invalid request + standardize curls

a7674c9


          Snippet before

a017493


          Dont import wasm in parent thread

906eccb


          Dont detect duplicates in webassembly/webservice.js,webassembly/opena…

f37019d

…pi.js

As they are mostly duplicates of webassembly/webservice-threaded.js


          Dont forget Node.JS


          Add pro/cons for each way

c639118

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved


          Apply suggestions from code review

f68f8fb

Co-authored-by: Faruk D. <fdiblen@users.noreply.github.com>

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved


          Update README.md

06f2987

Co-authored-by: Faruk D. <fdiblen@users.noreply.github.com>

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved


          Apply suggestions from code review

263900d

Co-authored-by: Faruk D. <fdiblen@users.noreply.github.com>

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved


          Update README.md

690b66c

Co-authored-by: Faruk D. <fdiblen@users.noreply.github.com>

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved

fdiblen reviewed

View reviewed changes

README.md Outdated Show resolved Hide resolved


          Apply suggestions from code review

2913dd5

Co-authored-by: Faruk D. <fdiblen@users.noreply.github.com>

fdiblen approved these changes

View reviewed changes

Collaborator

fdiblen left a comment

Awesome!

🥇


          Update README.md

3421b47

Co-authored-by: Faruk D. <fdiblen@users.noreply.github.com>

sverhoeven merged commit 06f5afe into master

sverhoeven deleted the develop branch

June 18, 2020 13:58

sonarqubecloud bot commented Jun 18, 2020

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
No Duplication information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment