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

Match file path relative to config file's directory (2) #853

Merged
merged 2 commits into from
Apr 12, 2021
Merged

Match file path relative to config file's directory (2) #853

merged 2 commits into from
Apr 12, 2021

Conversation

paulolieuthier
Copy link
Contributor

make test shows no errors.

@paulolieuthier paulolieuthier changed the title Fix rule relative file matching Match file path relative to config file's directory (2) Apr 9, 2021
@autrilla
Copy link
Contributor

Please make this merge onto develop, and rebase such that it's on top of the latest develop commit.

@paulolieuthier paulolieuthier changed the base branch from master to develop April 12, 2021 11:24
@paulolieuthier
Copy link
Contributor Author

Ready.

@autrilla autrilla merged commit c5b1298 into getsops:develop Apr 12, 2021
@paulolieuthier paulolieuthier deleted the fix-rule-relative-file-matching branch April 12, 2021 12:28
@paulolieuthier
Copy link
Contributor Author

Looking forward to the next release!

@rayterrill
Copy link

@paulolieuthier @autrilla This change appears to be breaking things see #1029. Is there a plan to revert this, or mitigate?

@paulolieuthier
Copy link
Contributor Author

Hey, @rayterrill thanks for pointing that out. I've commented on the issue, let's see where the maintainers will go with that.

simcax pushed a commit to simcax/sops that referenced this pull request Sep 23, 2023
* Match file path relative to config file's directory

* Update README.rst

Signed-off-by: Carsten Skov <carsten@simcax.dk>
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.

3 participants