-
-
Notifications
You must be signed in to change notification settings - Fork 1
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.
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
Javascript -- v1.2.0: Reimplement esm transpilation, and swap to it as the default. #479
Merged
Merged
Changes from all commits
Commits
Show all changes
149 commits
Select commit
Hold shift + click to select a range
e0ce087
Add links to the devlog
Skenvy acbb5e5
Add the default package json
Skenvy d5bb46c
Merge branch 'main' into javascript
Skenvy 1874a07
Merge branch 'main' into javascript
Skenvy 58bd634
Add more details to the package.json
Skenvy a6f7b90
A lot of comments in the devlog
Skenvy bc115c2
Update the workflows and add some basic recipes and remove the exit 1…
Skenvy 96ebaed
Add cache dependency path to subdir
Skenvy 9ab638e
Remove caching in setup-node
Skenvy b82f5a5
Add dist download and single publish recipe
Skenvy da8652d
Uncomment publish-npm
Skenvy ae6bad9
Remove the NA placeholder password in the java-test workflow, codeQL …
Skenvy 41d9c76
Merge branch 'main' into javascript
Skenvy 9b5ee75
Fix the missing permssions for the nested codeql step and add them to…
Skenvy 5ab3c1d
Merge branch 'main' into javascript
Skenvy b36cc65
Remove the trailing slash on the path, and add a release upload target
Skenvy 165d125
Add a word to the package
Skenvy 2867401
Merge branch 'main' into javascript
Skenvy 5255579
Whoops, remove the clean off publish
Skenvy d80db40
Bump the package.json to trigger package file changed
Skenvy e71f36c
Merge branch 'main' into javascript
Skenvy ff21c00
Add missing permission for publishing a package to github
Skenvy d046f7c
Merge branch 'main' into javascript
Skenvy 3118378
Add npm-gh reference in main readme
Skenvy 8d4938a
Add module dichotomy paragraph to devlog
Skenvy e9632eb
Merge branch 'main' into javascript
Skenvy 8711eda
Merge branch 'main' into javascript
Skenvy b5a2d7a
Add mention of typescript
99ec339
Added some typescript settings to run tsc for cjs and mjs
da486a0
Added mocha-chai test setup for unit test framework.
aacb7d9
Bump emojis
Skenvy b9bd9f5
Some further comments regarding the snyk guide
Skenvy ae56b7a
Bump version
20ad21c
Run pack to have the library up to date.
474a0a4
Merge branch 'main' into javascript
932d538
Merge branch 'main' into javascript
Skenvy 9f521d5
WIP fuse together snippets from the julia, java, and python implement…
64d01b9
Print the first layer of deps installed in ci
d86d59d
Change the minimum node version to 14.
48822a6
Update engine requires and install eslint
fb3dbe8
Add a linting step to CI (this push will fail on linting)
3dc78fd
Try adding AirBnB eslint config
24f9b9a
Fix test issue (lint will fail)
08340a1
Fewer linting errors
3b0eb60
Linter only warning now.
dd008ae
Format devlog
d508e6a
Add the reverse function
9d2793c
Tested and linted the reverse function
c2602fd
Add a recipe to fail if the checked in JS mismatches with the checked…
440b71b
Clean before the new verify
ebd3d00
Comment about docs gene
3674010
Install typedoc and tsdoc lint
ac99b63
Add tsdoc linter and try to make comments tsdoc-ish
70bb9ff
Recipe for docs
5eeae0a
Less cleaning when doccing
4516a92
Comment on attempt at JSDoc
Skenvy d685d32
Comment about setting up the docs target branch
f3e7d35
Bump up the version to test the whole CICD for javascript, with the f…
0288570
Retranspile the js files
2b87172
Add the devDependencies to the docs step
Skenvy 515fec8
Stop windows from corrupting the line endings that eslint requires
Skenvy c971d61
Merge branch 'main' into javascript
Skenvy c078d06
Merge branch 'main' into javascript
Skenvy c7c279e
Add words to the readme
d1edc65
yike the lock
2944619
Up the node version in CI to one that bumps npm over 8.6
0ba765c
version
57aa219
Merge branch 'main' into javascript
Skenvy a49b99c
Merge branch 'main' into javascript
b3a8d9e
npm up
9085688
Add stoppingTimeTerminus
086bf17
transpile
5770593
wip adding hailstone
4fbecdf
transpile
35dc605
Add make for http-server and comment regarding __namedParameters
57c82b0
Swap from a named interface to anonymous interfaces i.e. object liter…
2bb5004
line length
b5cbbfb
WIP tests for hailstone sequence
2153c0f
WIP tests for hailstone sequence
70f380c
Finished testing hailstones
d0ee908
Bump to 0.4.0
Skenvy d9b6e33
Merge branch 'main' into javascript
Skenvy 982d1c4
Stopping Time
21bb328
split into multiple files
9de2615
cutover to multiple files, tests and lints work
9e02a65
Add coverage reports being generated into docs
c14422f
coverage report handled
b5d22e8
pkg and lib
7576741
Merge branch 'main' into javascript
Skenvy e068ede
WIP tree graph
5a8a23e
assign the provided empty map in the test tgn's
89d7fcb
swap tgn to subtree
Skenvy 93c102b
Set up the deep equals test for tree graph
Skenvy 5a8904f
test, lint, and verify
Skenvy a63ba77
bump checkout
Skenvy b2c3de6
Add cache, add a basic stub for demos, and add comments regarding deno
Skenvy dfdb501
fix wf default and add node types to demo
Skenvy 2d953d3
fix name of needed wf job
Skenvy ddf96f1
dot the demo folder
Skenvy 8ef25e5
pkg
6275e19
Does setting up node with the development version fix npm not running…
Skenvy 113113a
ok, ls the dir first, but also see if strip version manually works
Skenvy 4a43ef9
but without another thing running first it doesn't run preinstall??
Skenvy f554f19
Does changing from preinstall to prepare let it know to run before th…
Skenvy 10c6ea3
Does using the full name 'install' fix it?
Skenvy 4ec9dd8
ok clearly lifecycle scripts are fucked
Skenvy 8066106
Add ts demos
fe3ffbc
remove spaces
7f6dcc9
bump package to first major version
34fde4c
Merge branch 'main' into javascript
Skenvy 002838f
Merge branch 'main' into javascript
Skenvy 1f2a3f1
Stop testing node on version 17 and 19
cbd0b4a
Merge branch 'main' into javascript
022eb13
Set new minor release to update allowed engines
db578bc
Add wraps in the make to preempt nvm use on every npm command
b0c0425
Add overwrite of the nvmrc for the test steps
3d963bf
put install recipes in conditional make vars to not use nvm on windows
bf0d331
Merge branch 'main' into javascript
Skenvy 8dd8255
Install nvm and npm versions before using make publish
ce7066b
Merge branch 'main' into javascript
Skenvy bfc5f87
Merge branch 'main' into javascript
Skenvy 6fb3992
Merge branch 'main' into javascript
Skenvy 19d82a9
Ran npm i on the unchanged package
Skenvy 420f022
Bump mocha, @types/mocha, @types/node
Skenvy 9c820c4
Bump everything except chai
Skenvy 62b89d6
Bump chai to latest 4.x
Skenvy 616dd0d
Merge branch 'main' into javascript
Skenvy 248f85a
Bump chai to latest
Skenvy 9dbd209
Add a node-options loader to use esm with mocha
Skenvy a91a610
wip trying different things to let chai v5
Skenvy 00a11a8
Remove the attempt to make chai v5 work
Skenvy 61297bf
very WIP, writing about this ridiculous ecosystem
Skenvy 197dea7
demo to assert imports of the project work
Skenvy 31b144e
Add back the previous copy of the transpiled result so there's less d…
Skenvy 034408a
Remove the src and tests changes to record what the errors were
Skenvy e65684f
Revert all the things done to fix issues to replay the errors
Skenvy 1dd284a
make build works
Skenvy e3f112f
added more specificity to the demo steps, wip mjs'ing and cjs'ing the…
Skenvy ce86c20
the project is buildable and the demo runs!
Skenvy 007574f
use esm tsconf for docs
Skenvy 65a93ef
fix ems -> esm and add use of mocha tsconf in covr
Skenvy 7326064
Update setup-node action
Skenvy 8cc3f60
update the version
Skenvy 56a0ea7
fixed the ems -> esm typo in the demo package
Skenvy c377197
Change the demo test to use the new npm script names
Skenvy 2f1294e
Add cross-env to let the test script continue to work on windows
Skenvy b1164e3
Add eol spec to gitattr to stop windows runners from not liking the n…
Skenvy 277773d
Does changing package json to autocrlf fix the windows runner not lik…
Skenvy 10f3f1a
Just don't bother with verifying the lib checkin on the matrix of run…
Skenvy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,4 @@ | ||
package-lock.json | ||
_/*.js | ||
_/*.mjs | ||
_/*.cjs |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,7 @@ | ||
{ | ||
"extension": ["ts"], | ||
"spec": "./**/*.spec.ts", | ||
"require": "ts-node/register" | ||
"require": "ts-node/register", | ||
"loader": "ts-node/esm", | ||
"es-module-specifier-resolution": "node" | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Check notice
Code scanning / CodeQL
Unused variable, import, function or class Note