Skip to content

Add repository condition to release job#34

Merged
vectorcmdr merged 1 commit intovectorcmdr:mainfrom
maniro-x:build-req
Apr 6, 2026
Merged

Add repository condition to release job#34
vectorcmdr merged 1 commit intovectorcmdr:mainfrom
maniro-x:build-req

Conversation

@maniro-x
Copy link
Copy Markdown
Contributor

@maniro-x maniro-x commented Apr 6, 2026

Added vectorcmdr/NMSE repo as a build requirement so it won't auto-build on forks

Pull Request

🧩 Summary

Prevents releases on forks by adding a repo requirement to the release job. It still allows manual triggers so that build artifacts can be still made for testing.

✅ Checklist

  • I have read the Contributor Guidelines in CONTRIBUTING.md
  • My changes follow the project style and conventions
  • I have not introduced external packages or libraries
  • [NA] I have added localisation to UI strings
  • [NA] I added/updated tests (if applicable)
  • [NA] All existing and new tests pass
  • [NA] I have updated documentation as needed

🧪 Testing

Tested that it does not make a release on a forked repo but cannot verify it on the main repo.

Added vectorcmdr/NMSE repo as a build requirement so it won't auto-build on forks
@vectorcmdr
Copy link
Copy Markdown
Owner

Thanks, a welcome change to simplify release dilution!

@vectorcmdr vectorcmdr merged commit 4f8b616 into vectorcmdr:main Apr 6, 2026
1 check passed
@maniro-x maniro-x deleted the build-req branch April 6, 2026 18:21
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