-
Notifications
You must be signed in to change notification settings - Fork 21
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
chore: sync files #150
chore: sync files #150
Conversation
8a0116b
to
50ccb9a
Compare
2cb7e41
to
e48ea87
Compare
0a5cd15
to
110fdca
Compare
db55902
to
ca68bfb
Compare
Signed-off-by: GitHub <[email protected]>
ca68bfb
to
71447a1
Compare
.github/workflows/deploy-docs.yaml
Outdated
@@ -40,4 +40,3 @@ jobs: | |||
with: | |||
token: ${{ secrets.GITHUB_TOKEN }} | |||
latest: ${{ github.event_name != 'pull_request_target' && github.ref_name == github.event.repository.default_branch }} | |||
mkdocs-requirements-txt: ./mkdocs-requirements.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This pr cannot be merged because of this line.
After merging autowarefoundation/autoware-github-actions#190, sync-files can be merged because mkdocs-requirements.txt becomes not necessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hsgwa
I issued the same pull request in the past. @kenji-miyake suggested me to add a function to customize project-dedicated mkdocs-requirements-txt.
autowarefoundation/autoware-github-actions#163
I think that CARET project has to take care of this sync-files, so that we have to exclude or suppress this deletion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
related: #175
Signed-off-by: Takayuki AKAMINE <[email protected]>
Skipped
Added
Changed
Not changed
Deleted
Not found