-
Notifications
You must be signed in to change notification settings - Fork 6
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
Updates after repository transfer #11
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should there also be an action for publishing to npm? (or will that be done manually?)
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v2 | ||
with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's safe to remove this work-around. It has been fixed for a while: actions/checkout#237 (comment).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But this is still not closed actions/checkout#299?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea, but it has been indicated that it is a duplicate of actions/checkout#237 and I think it should be closed.
@ptrus, did you just volunteer for that 🙂? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than that, it looks good!
1710454
to
ff4c416
Compare
ff4c416
to
ee8a39a
Compare
Also for the release process, but let's do that separately once we introduce a proper release process (will open an issue). |
No description provided.