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

Migrate from Jenkins -> GitHub Actions #71

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

emmabh
Copy link

@emmabh emmabh commented Sep 30, 2021

NB: had to lock in a version of gitdb2 that was compatible with GitPython

@emmabh emmabh marked this pull request as ready for review September 30, 2021 17:17
@emmabh emmabh requested a review from Didericis September 30, 2021 17:18
@emmabh emmabh requested a review from ssyberg September 30, 2021 17:18
- name: Install requirements from all requirements.txt files
run: |
INSTALL_COMMAND="pip install"
for file in sykle/plugins/**/requirements.txt; do
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 nice little bash snippet

Didericis
Didericis previously approved these changes Oct 12, 2021
@Didericis Didericis dismissed their stale review October 12, 2021 18:00

CD is off, forgot context

@@ -0,0 +1,15 @@
name: CD
Copy link
Contributor

Choose a reason for hiding this comment

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

@emmabh this can actually be removed; originally, when we had most of our projects on jenkins, this would ensure that a push to this repo would automatically update jenkins so that the internal syk command available on that jenkins instance was up to date. So the sudo make install command that was previously running on jenkins was just for that instance.

Copy link
Author

Choose a reason for hiding this comment

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

a-ha! that makes total sense, I was wondering why we had a cd that didn't go anywhere but it all makes sense now lol.

Copy link
Contributor

@Didericis Didericis left a comment

Choose a reason for hiding this comment

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

CD workflow can be removed/is no longer necessary, otherwise LGTM

@emmabh emmabh requested a review from Didericis October 14, 2021 15:06
@Didericis Didericis removed their assignment Sep 14, 2024
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