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

goja: bump version #2317

Merged
merged 1 commit into from
Jan 5, 2022
Merged

goja: bump version #2317

merged 1 commit into from
Jan 5, 2022

Conversation

codebien
Copy link
Contributor

@codebien codebien commented Jan 5, 2022 •
edited
Loading

It adds the support for:

  • optional chaining
  • error wrapping for interrupts

and fixes two issues with source maps

Commands executed:

$ go get github.com/dop251/goja@master
$ go mod tidy
$ go mod vendor

@codebien codebien added this to the v0.36.0 milestone Jan 5, 2022
@codebien codebien self-assigned this Jan 5, 2022
@github-actions github-actions bot requested review from na-- and yorugac January 5, 2022 16:46
@na--
Copy link
Member

na-- commented Jan 5, 2022

Can you do just go get github.com/dop251/goja@master, without the -u, to not update transitive dependencies but just goja?

@codebien
goja: bump version
4399887 
It includes the following changes:
	* support optional chaining
	* support error wrapping for interrupts
	* source maps fixes
@codebien
Copy link
Contributor Author

codebien commented Jan 5, 2022

@na-- done and executed also go mod tidy in the order as in the description

@codebien codebien merged commit 9d6adcc into master Jan 5, 2022
@codebien codebien deleted the update-goja branch January 5, 2022 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@na-- na-- na-- approved these changes

@mstoykov mstoykov mstoykov approved these changes

@yorugac yorugac Awaiting requested review from yorugac

Assignees

@codebien codebien

Labels
None yet
Projects
None yet
Milestone
v0.36.0
Development

Successfully merging this pull request may close these issues.
3 participants
@codebien @na-- @mstoykov