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

Tooling Improvements #6

Merged
merged 27 commits into from
Aug 14, 2023
Merged

Tooling Improvements #6

merged 27 commits into from
Aug 14, 2023

Conversation

sgammon
Copy link
Owner

@sgammon sgammon commented Aug 13, 2023

Summary

Adds more project documentation and prepares for full BCR support.

Constituent PRs

Task list

  • Working Bzlmod sub-module test
  • CI improvements
  • Docs
    • DCO, CLA, PR template
    • Contribution guide
    • Security policy
    • README improvements
    • Copy-paste samples updated
    • Actions job for deploying docs site
  • BCR Support
    • Config files for module publish
  • Tooling
    • Dependency review
    • Support for Bazel CI
    • Dependabot
    • Renovate
    • CodeQL
    • OpenSSF Scorecard
    • Hardened CI
    • Linting
    • Support for full-blown integration tests note: next pr
    • Release flow for BCR

Changelog

  • feat: add bazel ci presubmit
  • test: run integration test for bzlmod in ci
  • chore: add contribution guide
  • chore: readme updates
  • chore: doc link fixes
  • chore: add CLA, DCO docs
  • chore: add PR template
  • chore: add dependency review config
  • chore: drop outdated travis config
  • chore: readme updates

@sgammon sgammon added the enhancement New feature or request label Aug 13, 2023
@sgammon sgammon self-assigned this Aug 13, 2023
@sgammon sgammon force-pushed the chore/tooling branch 2 times, most recently from 67e5ac5 to 3d141cf Compare August 13, 2023 22:46
@CLAassistant
Copy link

CLAassistant commented Aug 13, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@alexeagle
Copy link

Hey, you should check out https://github.com/bazel-contrib/publish-to-bcr/ for the actual publish step, so that every release gets automatically published to the registry.

@sgammon
Copy link
Owner Author

sgammon commented Aug 13, 2023

@alexeagle heya :) will do -- should i set that up before i get approved on the BCR PR?

@alexeagle
Copy link

I generally do it from the beginning, following https://github.com/bazel-contrib/rules-template
If you do a manual publish, it's just delaying the release where you work out the automation.

@sgammon
Copy link
Owner Author

sgammon commented Aug 13, 2023

nice 👍🏻 caught me just in time, and on the perfect PR too. I'll add the .bcr configs and get the action going. I figured I'd have to be approved before I could push.

@sgammon sgammon force-pushed the chore/tooling branch 4 times, most recently from 179b97c to b68ab01 Compare August 14, 2023 00:24
@sgammon sgammon marked this pull request as ready for review August 14, 2023 00:25
sgammon and others added 14 commits August 13, 2023 17:27
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: StepSecurity Bot <bot@stepsecurity.io>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
- feat: add bazel ci presubmit
- test: run integration test for bzlmod in ci
- chore: add CLA, DCO docs
- chore: add PR template
- chore: add dependency review config
- chore: drop outdated travis config
- chore: readme updates

Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
- add java linter plugin
- add prettier-ignore and prettier-config
- add pnpm lock
- add husky
- add lint-staged
- add configuration to check lint before commit

Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
Signed-off-by: Sam Gammon <sam@elide.ventures>
@sgammon sgammon added dependencies Dependency updates and other related issues dev Development improvements and issues labels Aug 14, 2023
@sgammon sgammon merged commit 8471d14 into main Aug 14, 2023
20 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Dependency updates and other related issues dev Development improvements and issues enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants