Skip to content

Enable publishing #942

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

Merged
merged 1 commit into from
Nov 8, 2019
Merged

Enable publishing #942

merged 1 commit into from
Nov 8, 2019

Conversation

dcodeIO
Copy link
Member

@dcodeIO dcodeIO commented Nov 8, 2019

This PR is what's necessary to enable publishing to npm and gpr. Unfortunately, whether this will work or not is hard to test, so I'm wondering if we should just merge it and see what happens. If anything goes wrong, we can still learn from it and update to 0.9.0 or something. Objections?

@MaxGraey
Copy link
Member

MaxGraey commented Nov 8, 2019

probably to 0.8.1 not 0.9.0 =)

@dcodeIO
Copy link
Member Author

dcodeIO commented Nov 8, 2019

Or that :)

@dcodeIO
Copy link
Member Author

dcodeIO commented Nov 8, 2019

My guess so far is that npm publishing should work, but gpr fails for one of the following reasons:

  • Main package is named assemblyscript, not @AssemblyScript/assemblyscript
  • Loader package is named @assemblyscript/loader, not @AssemblyScript/loader or loader ?!
  • More than just GITHUB_TOKEN must be set up to enable proper publishing

Now, as chances are that at least npm will work, I think we can give this a try.

@dcodeIO dcodeIO merged commit a8d2755 into master Nov 8, 2019
@dcodeIO
Copy link
Member Author

dcodeIO commented Nov 8, 2019

Will know in about half an hour :). Or, wait... actually 2 1/2 hours?!

Edit: Now it should be in 10 minutes.

@dcodeIO dcodeIO deleted the publish-enable branch January 1, 2020 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants