Skip to content

Conversation

Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Nov 4, 2022

The GitHub actions have been updated to match the module template, with some exceptions. The docs publishing was omitted because this repository does not yet have API documentation, and I left some additional steps in the prepack script because they were present already.

@Gudahtt Gudahtt marked this pull request as ready for review November 4, 2022 23:03
@Gudahtt Gudahtt requested a review from a team as a code owner November 4, 2022 23:03
@Gudahtt Gudahtt marked this pull request as draft November 4, 2022 23:20
@Gudahtt Gudahtt force-pushed the update-github-actions branch from 13211f7 to 78f0cb5 Compare November 7, 2022 22:30
@Gudahtt
Copy link
Member Author

Gudahtt commented Nov 7, 2022

This depends upon #45 (for the lint error) and #46 (for the Node.js v19 error)

@Gudahtt Gudahtt force-pushed the update-github-actions branch from 78f0cb5 to 58c735a Compare November 8, 2022 01:01
The GitHub actions have been updated to match the module template, with
some exceptions. The docs publishing was omitted because this
repository does not yet have API documentation, and I left some
additional steps in the prepack script because they were present
already.
@Gudahtt Gudahtt force-pushed the update-github-actions branch from 58c735a to abe2472 Compare November 8, 2022 01:09
@Gudahtt Gudahtt marked this pull request as ready for review November 8, 2022 01:11
@Gudahtt Gudahtt merged commit 2771767 into main Nov 8, 2022
@Gudahtt Gudahtt deleted the update-github-actions branch November 8, 2022 15:33
Gudahtt added a commit that referenced this pull request Nov 9, 2022
The publishing instructions in the README have been updated to reflect
changes introduced in #44. We forgot to update the README in that PR.

The release instructions should now match those in the module template.
Gudahtt added a commit that referenced this pull request Nov 9, 2022
The publishing instructions in the README have been updated to reflect
changes introduced in #44. We forgot to update the README in that PR.

The release instructions should now match those in the module template.
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