Conversation
Signed-off-by: Sanket Sudake <sanketsudake@gmail.com>
Signed-off-by: Sanket Sudake <sanketsudake@gmail.com>
✅ Deploy Preview for fission-website ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Signed-off-by: Sanket Sudake <sanketsudake@gmail.com>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| * Restructure fission-bundle CLI and remove old modules by @sanketsudake in https://github.com/fission/fission/pull/3178 | ||
| * Bump the go-dependencies group across 1 directory with 2 updates by @dependabot[bot] in https://github.com/fission/fission/pull/3181 | ||
| * Update Go version to 1.24.1 by @sanketsudake in https://github.com/fission/fission/pull/3182 | ||
| * Bump github/codeql-action from 3.28.10 to 3.28.11 in the github-actions group by @dependabot[bot] in https://github.com/fission/fission/pull/3185 |
There was a problem hiding this comment.
[LanguageTool] reported by reviewdog 🐶
The official name of this software platform is spelled with a capital “H”. (GITHUB[1])
Suggestions: GitHub
URL: https://en.wikipedia.org/wiki/GitHub
Rule: https://community.languagetool.org/rule/show/GITHUB?lang=en-US&subId=1
Category: CASING
| * Bump the go-dependencies group across 1 directory with 8 updates by @dependabot[bot] in https://github.com/fission/fission/pull/3266 | ||
| * Add support to customise storagesvc deployment strategy by @ian-thompson-yv in https://github.com/fission/fission/pull/3196 | ||
| * Bump the docker-images group across 5 directories with 1 update by @dependabot[bot] in https://github.com/fission/fission/pull/3267 | ||
| * Bump github/codeql-action from 4.31.0 to 4.31.2 in the github-actions group by @dependabot[bot] in https://github.com/fission/fission/pull/3272 |
There was a problem hiding this comment.
[LanguageTool] reported by reviewdog 🐶
The official name of this software platform is spelled with a capital “H”. (GITHUB[1])
Suggestions: GitHub
URL: https://en.wikipedia.org/wiki/GitHub
Rule: https://community.languagetool.org/rule/show/GITHUB?lang=en-US&subId=1
Category: CASING
There was a problem hiding this comment.
Pull request overview
This PR updates documentation and configuration files for the v1.22.0 release of Fission, including version bumps for Hugo, Go, and project dependencies, along with comprehensive release notes documenting changes, fixes, and new contributors.
Key Changes
- Updated Hugo from 0.142.0 to 0.152.2 and Go from 1.23.5 to 1.25.x across build configurations
- Updated project dependencies including google/docsy (0.11.0 → 0.13.0) and bootstrap (5.3.3 → 5.3.8)
- Added comprehensive v1.22.0 release notes with deprecation notices, fixes, changelog, and contributor acknowledgments
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| netlify.toml | Updated HUGO_VERSION to 0.152.2 and GO_VERSION to 1.25.5 across all deployment contexts (production, split1, deploy-preview, branch-deploy) |
| go.mod | Updated Go version to 1.25.4 and google/docsy dependency to 0.13.0 |
| go.sum | Updated dependency checksums for Font-Awesome, google/docsy, and bootstrap to match go.mod changes |
| config.toml | Updated release_version from v1.21.0 to v1.22.0 |
| content/en/docs/releases/v1.22.0.md | Added new release notes document detailing deprecations, fixes, changelog with 83 PRs, and 4 new contributors |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| module github.com/fission/fission.io | ||
|
|
||
| go 1.23.5 | ||
| go 1.25.4 |
There was a problem hiding this comment.
The Go version in go.mod is set to 1.25.4, but netlify.toml specifies GO_VERSION as 1.25.5 across all deployment contexts. These versions should be consistent to avoid potential build inconsistencies between local development and deployment environments.
| go 1.25.4 | |
| go 1.25.5 |
| * Bump the go-dependencies group across 1 directory with 5 updates by @dependabot[bot] in https://github.com/fission/fission/pull/3232 | ||
| * Bump the github-actions group across 1 directory with 7 updates by @dependabot[bot] in https://github.com/fission/fission/pull/3233 | ||
| * Bump the go-dependencies group across 1 directory with 19 updates by @dependabot[bot] in https://github.com/fission/fission/pull/3238 | ||
| * Usa labels instead of deprecated commonLabels by @facorazza in https://github.com/fission/fission/pull/3239 |
There was a problem hiding this comment.
Typo: "Usa" should be "Use" - the word appears to be missing the final "e".
| * Usa labels instead of deprecated commonLabels by @facorazza in https://github.com/fission/fission/pull/3239 | |
| * Use labels instead of deprecated commonLabels by @facorazza in https://github.com/fission/fission/pull/3239 |
No description provided.