Skip to content

feat: Refactor plural up to no longer use git crypt#700

Merged
maciaszczykm merged 27 commits intomainfrom
marcin/prod-4388-refactor-plural-up-to-no-longer-use-git-crypt
Feb 11, 2026
Merged

feat: Refactor plural up to no longer use git crypt#700
maciaszczykm merged 27 commits intomainfrom
marcin/prod-4388-refactor-plural-up-to-no-longer-use-git-crypt

Conversation

@maciaszczykm
Copy link
Member

@maciaszczykm maciaszczykm commented Feb 6, 2026

Connected to pluralsh/bootstrap#124.

Summary

Labels

Test Plan

Checklist

  • If required, I have updated the Plural documentation accordingly.
  • I have added tests to cover my changes.
  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • I have added relevant labels to this PR to help with categorization for release notes.

@linear
Copy link

linear bot commented Feb 6, 2026

@maciaszczykm maciaszczykm added the enhancement New feature or request label Feb 6, 2026
@maciaszczykm maciaszczykm force-pushed the marcin/prod-4388-refactor-plural-up-to-no-longer-use-git-crypt branch from fde4dc4 to 7c8e2a1 Compare February 9, 2026 12:28
@maciaszczykm maciaszczykm marked this pull request as ready for review February 11, 2026 14:27

// CheckGitCrypt method checks if the .gitattributes and .gitignore files exist and have desired content.
// Some old repos can have fewer files to encrypt and must be updated.
func CheckGitCrypt(c *cli.Context) error {
Copy link
Member

Choose a reason for hiding this comment

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

I'm guessing this had no callers? I can't see where it's being removed

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, this was unused.

@maciaszczykm maciaszczykm merged commit 9db0ab9 into main Feb 11, 2026
14 checks passed
@maciaszczykm maciaszczykm deleted the marcin/prod-4388-refactor-plural-up-to-no-longer-use-git-crypt branch February 11, 2026 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants