Skip to content

Conversation

@dshemetov
Copy link
Contributor

No description provided.

Copy link
Contributor

@dsweber2 dsweber2 left a comment

Choose a reason for hiding this comment

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

Seems like something we'll only see the actual results of once it's merged, unfortunately, but seems sane

@dshemetov
Copy link
Contributor Author

dshemetov commented Feb 23, 2024

Something (I'm going to guess devtools::build_readme() or some related function) installed a Git Hook into my repo that complained if I did not commit both the md and rmd.

➜  epidatr git:(ds/readme) ✗ git add README.md 
➜  epidatr git:(ds/readme) ✗ git commit -m "as"
README.Rmd and README.md should be both staged
use 'git commit --no-verify' to override this check

So this CI is extra, but I figure it can't hurt. Also this CI will actually push a commit to the branch (before the PR merge), so it's a little more handy.

@dshemetov dshemetov merged commit cac4ca4 into dev Feb 23, 2024
@dshemetov dshemetov deleted the ds/ci branch February 23, 2024 19:48
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