Skip to content

Upgrade actions/checkout and best-of-lists action#352

Closed
sebastiaanspeck wants to merge 1 commit intomkdocs:mainfrom
sebastiaanspeck:patch-2
Closed

Upgrade actions/checkout and best-of-lists action#352
sebastiaanspeck wants to merge 1 commit intomkdocs:mainfrom
sebastiaanspeck:patch-2

Conversation

@sebastiaanspeck
Copy link

What kind of change does this PR introduce?

  • Add a project
  • Update a project
  • Remove a project
  • Add or update a category
  • Change configuration
  • Documentation
  • Other, please describe: CI

Description:

Checklist:

  • I have read the CONTRIBUTING guidelines.
  • I have not modified the README.md file. Projects are only supposed to be added or updated within the projects.yaml file since the README.md file is automatically generated.

@pawamoy
Copy link
Contributor

pawamoy commented Aug 31, 2025

Is this the result of a Dependabot run? Or did you do it manually? Why did you uncomment the release part? We don't want those. Also, we must keep using our own fork of best-of-generator.

Comment on lines -46 to +50
- name: install best-of-generator
run: pip install git+https://github.com/mkdocs/best-of-generator
- name: update-best-of-list
run: best-of generate projects.yaml --github-key=${{ secrets.GITHUB_TOKEN }} --libraries-key=${{ secrets.LIBRARIES_KEY }}
uses: best-of-lists/best-of-update-action@v0.8.5
with:
libraries_key: ${{ secrets.LIBRARIES_KEY }}
github_key: ${{ secrets.GITHUB_TOKEN }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Please revert, we must keep using our own fork.

Comment on lines -70 to +80
# - name: create-release
# uses: softprops/action-gh-release@v2
# env:
# GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# with:
# tag_name: ${{ env.VERSION }}
# name: "Update: ${{ env.VERSION }}"
# body_path: "latest-changes.md"
# draft: true
# prerelease: false
- name: create-release
uses: softprops/action-gh-release@v2
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
with:
tag_name: ${{ env.VERSION }}
release_name: "Update: ${{ env.VERSION }}"
body_path: "latest-changes.md"
draft: true
prerelease: false
Copy link
Contributor

Choose a reason for hiding this comment

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

Please revert, we don't want to make any release.

Copy link
Author

Choose a reason for hiding this comment

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

What is the reason behind this decision?

Copy link
Contributor

Choose a reason for hiding this comment

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

They are not useful (it's just a catalog, not an installable library or app), and very noisy (they appear in the home feed) 🙂

Copy link
Author

@sebastiaanspeck sebastiaanspeck Aug 31, 2025

Choose a reason for hiding this comment

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

Then maybe also disable/remove those branches/tags as well 😄 to me it looked like it was a semi-finished product

Copy link
Contributor

Choose a reason for hiding this comment

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

It's useful to keep it commented in case someone wants to re-enable them in the future 🤷

@sebastiaanspeck sebastiaanspeck deleted the patch-2 branch August 31, 2025 17:58
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