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

Cross compatible #114

Merged
merged 1 commit into from
Dec 29, 2021
Merged

Cross compatible #114

merged 1 commit into from
Dec 29, 2021

Conversation

shanedell
Copy link
Collaborator

@shanedell shanedell commented Dec 29, 2021

Multiple Updates:

  • Create module/index.js that will be used for running the package
  • Make the node addon file have the OS name tagged on the end
  • Update README.md
  • Update bindings for newer version of nodejs
  • Update JavaScript example files to use module/omega_edit
  • Created a release.yml that will create a release tarball of the current code and upload it to gitlab
    • Also create a npm package.

@scholarsmate If you don't have an account already, create an account for npmjs.com and send me the username so I can add you to omega-edit package.

After this you will need to create an access token and save it as GitHub Secret named NPM_TOKEN.

This will be used for when a tag is pushed the omega-edit package will be published.

Here is release built by the CI https://github.com/Shanedell/omega-edit/releases/tag/v0.6.1-1. Here is the CI job for that release https://github.com/Shanedell/omega-edit/actions/runs/1632789428. I was able to successfully install omega-edit@0.6.1-1 and can run it should also be able to do it from Linux, plan to work on the windows one in a separate PR

@shanedell shanedell changed the base branch from cross-compatible to main December 29, 2021 02:55
@shanedell shanedell marked this pull request as ready for review December 29, 2021 03:19
Copy link
Member

@scholarsmate scholarsmate left a comment

Choose a reason for hiding this comment

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

LGTM

@scholarsmate
Copy link
Member

The changes seem reasonable to me. I haven't yet got an npmjs.com account. Will get to that shortly and send you the details.

- Create module/index.js that will be used for running the package
- Make the node addon file have the OS name tagged on the end
- Update README.md
- Update bindings for newer version of nodejs
- Create release CI to create release and upload tarball file to github
  - Also publishes to the npm regsitry
@shanedell shanedell merged commit 9bb23c8 into ctc-oss:main Dec 29, 2021
@shanedell shanedell deleted the cross-compatible branch December 29, 2021 18:06
@scholarsmate scholarsmate added this to the v0.7.0 milestone Jan 11, 2022
@scholarsmate scholarsmate added the requirement This is a requirement label Jan 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requirement This is a requirement
Projects
Development

Successfully merging this pull request may close these issues.

2 participants