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

docs: Add new "arrows" favicon #1200

Merged
merged 29 commits into from
Jun 20, 2024
Merged

docs: Add new "arrows" favicon #1200

merged 29 commits into from
Jun 20, 2024

Conversation

DaveSkender
Copy link
Owner

@DaveSkender DaveSkender commented Jun 1, 2024

Still to do:

- [ ] test/tweak formatting in dev build, then copy to to deploy also
- [ ] confirm -1 syntax, we want to go back to prior tag, not prior commit in history
- [ ] see if chore and docs commits can be excluded
- [ ] post to build summary also, but without URL

  • remove PR coverage comments but keep in build summary, just annoying (same in Py proj)
  • update favicons

Repository owner deleted a comment from github-actions bot Jun 20, 2024
@DaveSkender
Copy link
Owner Author

this if far too fragile and unpredictable; and uneditable after published; reverting (bad idea)

@DaveSkender DaveSkender changed the title ci: Inject package release notes docs: Add new "arrows" favicon Jun 20, 2024
@DaveSkender DaveSkender self-assigned this Jun 20, 2024
@DaveSkender DaveSkender marked this pull request as ready for review June 20, 2024 06:26
@DaveSkender DaveSkender merged commit a18129b into main Jun 20, 2024
19 checks passed
@DaveSkender DaveSkender deleted the inject-release-notes branch June 20, 2024 06:28
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant