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

Set up CI for repo using github actions #170

Merged
merged 1 commit into from
Dec 17, 2019

Conversation

amisevsk
Copy link
Contributor

What does this PR do?

Add CI for pull requests that

  1. Runs shellcheck on all scripts
  2. Builds the registry
  3. Builds an offline registry

To test, I added it to my fork

What issues does this PR fix or reference?

Fixes eclipse-che/che#14867


jobs:
shellcheck:
runs-on: ubuntu-latest
Copy link
Contributor

Choose a reason for hiding this comment

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

could we do this on RHEL8 instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's a GitHub thing, your options are mac, windows, or ubuntu 🙁

I could look into updating it to run in a rhel docker container, but this is my first go at configuring GitHub actions.

Copy link
Contributor

Choose a reason for hiding this comment

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

+1 for this PR, +2 if you want to do a followup.

Copy link
Contributor

@nickboldt nickboldt left a comment

Choose a reason for hiding this comment

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

+1 despite the ubuntu flavour being different from Alpine or RHEL.

Signed-off-by: Angel Misevski <amisevsk@redhat.com>
@amisevsk amisevsk merged commit 4a3e20d into eclipse-che:master Dec 17, 2019
@amisevsk
Copy link
Contributor Author

What's left on this issue is adding branch protections in the settings tab on the devfile registry; I don't have access to this so someone else will have to do it.

@nickboldt
Copy link
Contributor

Just checked: don't have perms to view https://github.com/eclipse/che-devfile-registry/settings either. Maybe @l0rd or @benoitf have permission?

sparkoo pushed a commit to sparkoo/che-devfile-registry that referenced this pull request Dec 4, 2020
* Update omnisharp-theia-plugin to 0.0.2

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>

* Fix firstPublicationDate in latest omnisharp plugin

Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
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.

Set up CircleCI for devfile registry repo
2 participants