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

feat: now generate codeowners checks in user root but also repository root folder for .sauced.yaml #107

Merged
merged 8 commits into from
Aug 29, 2024

Conversation

nickytonline
Copy link
Member

@nickytonline nickytonline commented Aug 28, 2024

Description

Node the generate codeowners command checks for .sauced.yaml in ~/.sauced.yaml and opts.path + "/.sauced.yaml".

Related Tickets & Documents

Closes #102

Mobile & Desktop Screenshots/Recordings

Steps to QA

  1. Run just install
  2. Run pizza generate codeowners some/path.
    a. Try without a some/path/.sauced.yaml file (fails)
    b. Try with a some/path/.sauced.yaml file (generates CODEOWNERS file)

Tier (staff will fill in)

  • Tier 1
  • Tier 2
  • Tier 3
  • Tier 4

[optional] What gif best describes this PR or how it makes you feel?

@nickytonline nickytonline force-pushed the nickytonline/autodetect-sauced-yaml branch from ac7ba39 to 10468ff Compare August 28, 2024 21:30
@nickytonline nickytonline marked this pull request as ready for review August 28, 2024 22:32
go.mod Show resolved Hide resolved
@nickytonline nickytonline requested a review from a team August 28, 2024 22:38
go.mod Show resolved Hide resolved
cmd/generate/codeowners/codeowners.go Outdated Show resolved Hide resolved
pkg/config/config.go Outdated Show resolved Hide resolved
pkg/config/config_test.go Show resolved Hide resolved
pkg/config/config_test.go Outdated Show resolved Hide resolved
pkg/config/config_test.go Outdated Show resolved Hide resolved
@nickytonline nickytonline force-pushed the nickytonline/autodetect-sauced-yaml branch from 06d1eca to c833602 Compare August 29, 2024 00:01
@nickytonline nickytonline force-pushed the nickytonline/autodetect-sauced-yaml branch from c833602 to aba6310 Compare August 29, 2024 00:03
go.mod Show resolved Hide resolved
pkg/config/config_test.go Outdated Show resolved Hide resolved
@nickytonline nickytonline merged commit 2959d12 into beta Aug 29, 2024
9 checks passed
@nickytonline nickytonline deleted the nickytonline/autodetect-sauced-yaml branch August 29, 2024 14:00
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.

Feature: Automatically detect in root of repository path for .sauced.yaml
3 participants