Skip to content

Switched to using dev version of fledge due to better messaging - #37

Open
aylapear wants to merge 3 commits into
mainfrom
fledge-dev-news
Open

Switched to using dev version of fledge due to better messaging #37
aylapear wants to merge 3 commits into
mainfrom
fledge-dev-news

Conversation

@aylapear

@aylapear aylapear commented Jul 20, 2026

Copy link
Copy Markdown
Member

The CRAN version of fledge does not pick up the titles of PR for their news file which as been our system and seems to work well, otherwise it uses the -/* notation and it doesn't seem people are in that habit.

Had to add pak install since not doing a CRAN version.

Choose to pin the version of fledge used, this means that it won't auto update but also means we know exactly that the current version is working and don't have to working about a changing breaking things.

Wasn't able to test fully but the plan would be to do a v2 version of these actions and test it on a select few before updating all the fledge actions that are in current use.

@aylapear
aylapear requested a review from joethorley as a code owner July 20, 2026 23:41
@aylapear
aylapear requested a review from sebdalgarno July 22, 2026 16:59
@aylapear

Copy link
Copy Markdown
Member Author

@sebdalgarno since Joe is away and I am not sure when he will look at GitHub I figured I could get you to review for now and we can merge in he doesn't chime in within a week or so. Happy to explain the changes if you need as it took me a bit to understand how the system is designed and works.

@sebdalgarno sebdalgarno left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

apparently there is 60/hour limit on unauthenticated installs from public repos. If add GITHUB_PAT for dev installation with pak then would never fail
i.e.,

env:
          GITHUB_PAT: ${{ steps.app-token.outputs.token }}

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