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.

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

Add publish script and workflow #4

Merged
merged 2 commits into from
Feb 20, 2024
Merged

Conversation

venetak
Copy link

@venetak venetak commented Feb 14, 2024

  • Added a script that will check if the version in the local package.json is the same as the one in npm. If it is different - it publishes the package.
  • Added a workflow that packs react-scripts-cothml and cra-template-cohtml and publishes them if needed on each push in main.

@venetak venetak requested review from Tony-CL and alien1976 February 14, 2024 15:37
@venetak venetak self-assigned this Feb 14, 2024
Comment on lines 20 to 21
- name: Setup Chrome
uses: browser-actions/[email protected]

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to setup Chrome for this? I am not that familiar with the actions so just wandering.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, we don't, I copy-pasted too much. The checkout action does not need Chrome.

@@ -0,0 +1,43 @@
# This workflow will do a clean install of node dependencies, build the source code and run tests across different versions of node

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not able to see running tests in this action :D

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copy-paste :D

@venetak venetak merged commit 0f8afaa into main Feb 20, 2024
0 of 10 checks passed
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.

3 participants