Skip to content

Conversation

@cpanato
Copy link
Member

@cpanato cpanato commented Dec 12, 2025

No description provided.

@cpanato cpanato force-pushed the add-bump-go-version branch from 5ed78f8 to 25b84fc Compare December 12, 2025 08:44
@cpanato cpanato requested a review from jml December 12, 2025 08:45
@cpanato cpanato force-pushed the add-bump-go-version branch from 25b84fc to cdacdd7 Compare December 12, 2025 08:48
Copy link

@jml jml left a comment

Choose a reason for hiding this comment

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

This is great! Sorry for the delay in reviewing it.

token: ${{ secrets.GITHUB_TOKEN }}
```

### Advanced Usage
Copy link

Choose a reason for hiding this comment

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

Please add a sentence or short paragraph saying what the point of this advanced usage is.


## Usage

### Basic Usage
Copy link

Choose a reason for hiding this comment

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

Please add a sentence or short paragraph saying what this basic usage gets you. That is, what happens when you run bump-go-version-file like this?

@@ -0,0 +1,39 @@
name: Bump Go Version
Copy link

Choose a reason for hiding this comment

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

Please add a comment describing what this workflow does for you at a high level.


steps:
- name: Checkout repository
uses: actions/checkout@v4
Copy link

Choose a reason for hiding this comment

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

Suggested change
uses: actions/checkout@v4
uses: actions/checkout@v6

Comment on lines +84 to +86
LATEST=$(curl -s https://go.dev/VERSION?m=text | head -n1 | sed 's/go//')
echo "latest=$LATEST" >> $GITHUB_OUTPUT
echo "Latest Go version: $LATEST"
Copy link

Choose a reason for hiding this comment

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

Please fix the actionlint here.

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