- Notifications
You must be signed in to change notification settings - Fork15
Add GitHub Actions pipeline for pgSphere#69
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
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.
I love it! Fantastic contribution!
esabol commentedSep 26, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
I also dream of a GitHub Actions pipeline which, on merges to the master branch, uploads the latest documentation to the GitHub Pages associated with this repository. Could you help with that? That's probably a separate pipeline though. |
@borodun: Actually, I think "make test" should come before "make install"? Also, I am curious as to why you chose |
@esabol, I moved |
@esabol, regarding the pipeline for docs: that’s the plan. We need to organize access to GitHub Pages for this repository. This is a more administrative issue. We want to get this done as soon as possible. |
The new GitHub Actions pipeline doesn't seem to be working in PR#65, unfortunately. Very weird error that I've not encountered before in my GitHub Actions. |
Changing the '/' to a '-' in https://github.com/mbiagetti/github-action-poc/pull/2/files But that method seemsvery convoluted and annoying. Would this method work? |
We can use something likehttps://github.com/marketplace/actions/github-slug-action to create slugged versions of variables. We can add:
And change I'll create PR |
Use GitHub Actions for pgSphere pipeline. At this moment, builds with HEALPix are failing due to compilation warning, but#68 should fix this.