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

Refactor CI #28

Merged
merged 2 commits into from
Jan 16, 2024
Merged

Refactor CI #28

merged 2 commits into from
Jan 16, 2024

Conversation

antoniovazquezblanco
Copy link
Contributor

This PR refactors the CI of the project:

  • Split the process into building natives, building the plugin and releasing
  • Build the plugin for all the supported versions of Ghidra, this acts as testing but also provides builds for everyone...
  • Use maintained github actions as much as possible to avoid maintaining steps such as downloading and installing Ghidra

Hope it helps!

- Avoid duplicate runs when creating a tag
- Switch to manual releases
@al3xtjames
Copy link
Owner

Thanks for cleaning this up! Overall this looks good, though I ended up making some changes. I'm OK with manually creating releases. It's probably better than my current approach of creating a release for each new commit on master. I'm less certain about supporting older versions of Ghidra - there have been some API changes previously (such as #15 and #20). 10.2 is likely recent enough though, so I'll keep it enabled in CI.

@al3xtjames al3xtjames merged commit 3f57834 into al3xtjames:master Jan 16, 2024
14 checks passed
@antoniovazquezblanco
Copy link
Contributor Author

Thank you!

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