Skip to content
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

Merge 1.3 branch in as v2.0.0 #256

Merged
merged 91 commits into from
May 28, 2022
Merged

Merge 1.3 branch in as v2.0.0 #256

merged 91 commits into from
May 28, 2022

Conversation

hildjj
Copy link
Contributor

@hildjj hildjj commented May 28, 2022

Version 2.0.0 complete.

Mingun and others added 30 commits October 18, 2021 00:52
@Mingun
source-map: Create SourceNodes for functions and initializers
769a432
@Mingun
source-map: Generate tables of constants as a SourceNode
fcae22a
@Mingun
source-map: Remove excess .join("\n")s
de0ad72 
Such joins stringify SourceNode's prematurally
@Mingun
source-map: Do not join arrays prematurely
4ad80de 
Again, this prevents premature stringification of the SourceNodes
@Mingun
source-map: Remove yet another premature join
9bdf8eb 
Here just for uniformity
@Mingun
source-map: Remove useless function
fa5da47
@Mingun
source-map: Generate top-level as a SourceNode
d482541
@Mingun
source-map: Generate module wrappers as a SourceNode
6eece33
@Mingun
source-map: Represent generated code in the AST as a SourceNode
8b7d69d
@Mingun
source-map: Don't Repeat Yourself - introduce new interface SourceOpt…
b352e92 
…ionsBase
@Mingun
source-map: Implement API for producing source maps
c08e4d8 
Regenerate parser, add documentation and tests
@Mingun
source-map: Run CLI tests in the test/cli directory
cadc935
@Mingun
source-map: Do not break process until generated code/source map will…
ef9f014 
… be written

Also exit with error code 2 when CLI fails because of invalid test input
and update CLI tests to check exit code
@Mingun
source-map: Add new -m/--source-map parameter to the CLI
e47b5f4
@Mingun
source-map: Introduce helper for generate a parse rule function name
e5b1ebf
@Mingun
source-map: Add mapping for the rule names to the names of the parse …
1ee49ab 
…functions
@Mingun
source-map: Cover ast2SourceNode completely
fe39cbf
@Mingun
source-map: Alternative API: new output mode "source-and-map" instead…
6f57b4a 
… of `sourceMap` flag
@hildjj @Mingun
source-map: Make source-map work in the browser
4ee3472
@Mingun
Update all dependencies
0ac78fd
@hildjj @Mingun
Only run lint on latest node, rather than holding back eslint to an o…
53f3d12 
…lder version or dropping node 10 support.
@Mingun
Fix a regression from fad4ab7 in the description of character classes
bd3798d 
Character class description contains redundant commas between class parts:

[1,2,3-5] instead of [123-5] for class [123-5]
@hildjj
Add example grammars for XML and source-mapping:mapping
f711f5b
@hildjj
Merge pull request #196 from hildjj/more-examples
04eb7b0 
Add example grammars for XML and source-mapping:mapping
@hildjj
Merge pull request #163 from Mingun/source-map
749913a 
Source map support
@hildjj
Merge pull request #197 from Mingun/fix-message-regression
8bc052c 
Fix a regression in the description of character classes in error message
@hildjj
Merge branch 'main' into 1.3
091b64b 
* main:
  Update link to minified code.  TODO: automate
@hildjj
Add browser-sync
f581acc
@hildjj
Move from cp to copyfiles. Fixes #198.
38f8e54
@hildjj
Don't lint leftover example js files
8057ffb
hildjj added 23 commits May 21, 2022 11:54
@hildjj
Merge pull request #215 from hildjj/test-issues
7c19b45 
Test issues
@hildjj
Make sourcemaps work by linking the URL to the map into the source fi…
813d7f4 
…le, or inlining it if requested.
@hildjj
Handle hidden maps
ddf6af8
@hildjj
Update CLI docs
041cc18
@hildjj
Merge pull request #218 from hildjj/sourcemap-url
92b937e 
Sourcemap url
@hildjj
Update dependency versions
c3c9537
@hildjj
Minor fixups to work with new dependencies
9200a46
@hildjj
Test on node 18 instead of 17
1f63370
@hildjj
Remove node 10 support
8b0be11
@hildjj
Switch to package-lock.json format to lockfileVersion 2
611b0c1
@hildjj
Merge pull request #248 from hildjj/update-deps
5a25b79 
Update deps
@hildjj
Move commander and source-map-generator to full dependencies
3a3be38
@hildjj
Use commander as dependency
e0eccad
@hildjj
Clean up bin/.eslintrc.json
2212eaa
@hildjj
Merge pull request #251 from hildjj/add-deps
bc388f2 
Add source-map-generator and commander as full dependencies
@hildjj
Update dependencies for web tests
941c79d
@hildjj
Deal with most of the rollup warnings by ensuring that node builtins …
4a33995 
…act as no-ops.
@hildjj
Ignore the sinon circular dependency warning. Fixes #220.
8eeaee8
@hildjj
Merge pull request #252 from hildjj/rollup-warnings
9a45172 
Rollup warnings
@hildjj
Update changelog.
d5b0b8c
@hildjj
Merge pull request #254 from hildjj/release-notes
c283a53 
Update changelog.
@hildjj
Update version to 2.0.0. Update copyright dates.
b91e6a1
@hildjj
Merge pull request #255 from hildjj/version
26e79ca 
Version
@hildjj hildjj merged commit edec7a0 into main May 28, 2022
@hildjj hildjj deleted the 1.3 branch May 28, 2022 17:07
@Mingun
Copy link
Member

Mingun commented Jun 1, 2022

@hildjj, could you restore 1.3 branch? Then it would be possible for me to retarget PRs to a main branch, as explained here. Then it can be deleted again :). I prefer to keep discussions about proposed features in single PR.

@hildjj hildjj restored the 1.3 branch June 1, 2022 16:34
@hildjj
Copy link
Contributor Author

hildjj commented Jun 1, 2022

Done. It turns out I hadn't deleted 1.3 from my own repo, so I checked that out, then pushed it upstream.

@Mingun
Copy link
Member

Mingun commented Jun 1, 2022

Nice. Probably need to reopen PRs -- I cannot do that myself. Could you please reopen #206, #208 and #240?

@hildjj
Copy link
Contributor Author

hildjj commented Jun 1, 2022

I got #206 and #240, but #208 has a problem, annotated there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hildjj @Mingun @cmfcmf