-
Notifications
You must be signed in to change notification settings - Fork 153
chore(master): release execution 1.5.0 #1207
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
base: master
Are you sure you want to change the base?
chore(master): release execution 1.5.0 #1207
Conversation
Note Reviews pausedUse the following commands to manage reviews:
""" WalkthroughA new minor release (1.5.0) for the "execution" component was documented in the changelog, introducing a feature for resolving fetch dependencies and two bug fixes: disabling test cache for execution tests and fixing representation collection for fetches scoped to concrete types. The version was updated in the release manifest. No changes were made to public APIs or exported entities. Changes
Possibly related PRs
Comment |
5f54ac9
to
c986c7e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
execution/CHANGELOG.md (1)
3-5
: Trim redundant blank line after the version headerTwo consecutive blank lines break the otherwise consistent spacing used throughout the changelog. Dropping one keeps the style uniform and silences the LanguageTool warning.
## [1.4.1](https://github.com/wundergraph/graphql-go-tools/compare/execution/v1.4.0...execution/v1.4.1) (2025-07-08) - ### Bug Fixes
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
execution/CHANGELOG.md
(1 hunks)release-please-manifest.json
(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- release-please-manifest.json
🧰 Additional context used
🪛 LanguageTool
execution/CHANGELOG.md
[grammar] ~3-~3: Use proper spacing conventions.
Context: .../v1.4.0...execution/v1.4.1) (2025-07-08) ### Bug Fixes * execution tests, disable te...
(QB_NEW_EN_OTHER_ERROR_IDS_000007)
[grammar] ~6-~6: Use proper spacing conventions.
Context: ...ion/v1.4.1) (2025-07-08) ### Bug Fixes * execution tests, disable test cache for ...
(QB_NEW_EN_OTHER_ERROR_IDS_000007)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Build and test (go 1.23 / windows-latest)
c986c7e
to
aed7bd8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
execution/CHANGELOG.md (1)
6-10
: Refine bullet phrasing for consistency & clarityThe two new bullet points break the existing style used elsewhere in this changelog (imperative verb without an interior comma) and trip
LanguageTool
’s spacing rule.
Consider:-* execution tests, disable test cache for execution -* fix collecting representation for fetches scoped to concrete types +* execution: disable test cache for execution tests +* fix collection of representations for fetches scoped to concrete types
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
execution/CHANGELOG.md
(1 hunks)release-please-manifest.json
(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- release-please-manifest.json
🧰 Additional context used
🪛 LanguageTool
execution/CHANGELOG.md
[grammar] ~3-~3: Use proper spacing conventions.
Context: .../v1.4.0...execution/v1.4.1) (2025-07-08) ### Bug Fixes * execution tests, disable te...
(QB_NEW_EN_OTHER_ERROR_IDS_000007)
[grammar] ~6-~6: Use proper spacing conventions.
Context: ...ion/v1.4.1) (2025-07-08) ### Bug Fixes * execution tests, disable test cache for ...
(QB_NEW_EN_OTHER_ERROR_IDS_000007)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Build and test (go 1.23 / windows-latest)
e833dfb
to
b79cbc9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (3)
execution/CHANGELOG.md (3)
8-9
: Tweak wording for smoother reading“implement resolving fetch dependencies” is slightly awkward. Consider:
-* implement resolving fetch dependencies … +* implement resolution of fetch dependencies …Keeps the style consistent with the other bullets while reading more naturally.
13-13
: Prefer colon over comma after subjectThe comma after “execution tests” makes the line a bit clunky; a colon is clearer:
-* execution tests, disable test cache for execution … +* execution tests: disable test cache for execution …Purely stylistic but improves readability.
4-5
: One blank line is enoughThere are two consecutive blank lines between the version header and “### Features”. Dropping one keeps the spacing consistent with previous sections.
- -Tiny cosmetic fix; feel free to ignore if release-please insists on the current formatting.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
execution/CHANGELOG.md
(1 hunks)release-please-manifest.json
(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- release-please-manifest.json
🧰 Additional context used
🪛 LanguageTool
execution/CHANGELOG.md
[grammar] ~3-~3: There might be a problem here.
Context: ...re/execution/v1.4.0...execution/v1.5.0) (2025-07-09) ### Features * implement resolving fetch dependencies ...
(QB_NEW_EN_MERGED_MATCH)
[grammar] ~8-~8: There might be a mistake here.
Context: ...dergraph/graphql-go-tools/issues/1216)) (ca9ebaa) ### Bug Fixes * execution tests, disable t...
(QB_NEW_EN_OTHER)
[grammar] ~11-~11: Use correct spacing
Context: ...da89c78239f83a1c3eba909b838)) ### Bug Fixes * execution tests, disable test cache for...
(QB_NEW_EN_OTHER_ERROR_IDS_5)
[grammar] ~14-~14: Use correct spacing
Context: ...dergraph/graphql-go-tools/issues/1200)) (bcf547d) ## [1.4.0](https://github.com/wundergraph/g...
(QB_NEW_EN_OTHER_ERROR_IDS_5)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Build and test (go 1.23 / windows-latest)
b79cbc9
to
178ce9d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
execution/CHANGELOG.md (2)
6-9
: Polish the feature line wording for clarity“implement resolving fetch dependencies” reads awkwardly. Consider a noun-phrase after “implement”:
-* implement resolving fetch dependencies (#1216) +* implement fetch-dependency resolution (#1216)This keeps the imperative style used elsewhere while eliminating the gerund stack.
11-14
: Tighten phrasing of bug-fix bulletsMinor rewording improves flow and mirrors the style used in prior entries:
-* execution tests, disable test cache for execution (#1206) -* fix collecting representation for fetches scoped to concrete types (#1200) +* disable test cache for execution tests (#1206) +* fix representation collection for fetches scoped to concrete types (#1200)No functional impact, just readability.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
execution/CHANGELOG.md
(1 hunks)release-please-manifest.json
(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- release-please-manifest.json
🧰 Additional context used
🪛 LanguageTool
execution/CHANGELOG.md
[grammar] ~3-~3: There might be a problem here.
Context: ...re/execution/v1.4.0...execution/v1.5.0) (2025-07-09) ### Features * implement resolving fetch dependencies ...
(QB_NEW_EN_MERGED_MATCH)
[grammar] ~8-~8: There might be a mistake here.
Context: ...dergraph/graphql-go-tools/issues/1216)) (ca9ebaa) ### Bug Fixes * execution tests, disable t...
(QB_NEW_EN_OTHER)
[grammar] ~11-~11: Use correct spacing
Context: ...da89c78239f83a1c3eba909b838)) ### Bug Fixes * execution tests, disable test cache for...
(QB_NEW_EN_OTHER_ERROR_IDS_5)
[grammar] ~14-~14: Use correct spacing
Context: ...dergraph/graphql-go-tools/issues/1200)) (bcf547d) ## [1.4.0](https://github.com/wundergraph/g...
(QB_NEW_EN_OTHER_ERROR_IDS_5)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Build and test (go 1.23 / windows-latest)
@coderabbitai pause |
✅ Actions performedReviews paused. |
@coderabbitai configuration |
CodeRabbit configurationlanguage: en-US
tone_instructions: ''
early_access: false
enable_free_tier: true
reviews:
profile: chill
request_changes_workflow: false
high_level_summary: true
high_level_summary_placeholder: '@coderabbitai summary'
high_level_summary_in_walkthrough: false
auto_title_placeholder: '@coderabbitai'
auto_title_instructions: >-
PR title follows the Conventional Commits specification, using the
appropriate type (e.g., feat, fix, docs) and scope. Examples: fix(cli):
upgrade dependency, feat(router): implement mcp server
review_status: true
commit_status: true
fail_commit_status: false
collapse_walkthrough: false
changed_files_summary: true
sequence_diagrams: false
assess_linked_issues: true
related_issues: true
related_prs: true
suggested_labels: false
auto_apply_labels: false
suggested_reviewers: false
auto_assign_reviewers: false
poem: false
labeling_instructions: []
path_filters: []
path_instructions: []
abort_on_close: true
disable_cache: false
auto_review:
enabled: true
auto_incremental_review: true
ignore_title_keywords: []
labels: []
drafts: false
base_branches: []
finishing_touches:
docstrings:
enabled: true
unit_tests:
enabled: true
tools:
ast-grep:
rule_dirs: []
util_dirs: []
essential_rules: true
packages: []
shellcheck:
enabled: true
ruff:
enabled: true
markdownlint:
enabled: true
github-checks:
enabled: true
timeout_ms: 90000
languagetool:
enabled: true
enabled_rules: []
disabled_rules: []
enabled_categories: []
disabled_categories: []
enabled_only: false
level: default
biome:
enabled: true
hadolint:
enabled: true
swiftlint:
enabled: true
phpstan:
enabled: true
level: default
phpmd:
enabled: true
phpcs:
enabled: true
golangci-lint:
enabled: true
yamllint:
enabled: true
gitleaks:
enabled: true
checkov:
enabled: true
detekt:
enabled: true
eslint:
enabled: true
flake8:
enabled: true
rubocop:
enabled: true
buf:
enabled: true
regal:
enabled: true
actionlint:
enabled: true
pmd:
enabled: true
cppcheck:
enabled: true
semgrep:
enabled: true
circleci:
enabled: true
clippy:
enabled: true
sqlfluff:
enabled: true
prismaLint:
enabled: true
pylint:
enabled: true
oxc:
enabled: true
shopifyThemeCheck:
enabled: true
luacheck:
enabled: true
brakeman:
enabled: true
dotenvLint:
enabled: true
htmlhint:
enabled: true
checkmake:
enabled: true
chat:
auto_reply: true
integrations:
jira:
usage: auto
linear:
usage: auto
knowledge_base:
opt_out: false
web_search:
enabled: true
code_guidelines:
enabled: true
filePatterns: []
learnings:
scope: auto
issues:
scope: auto
jira:
usage: auto
project_keys: []
linear:
usage: auto
team_keys: []
pull_requests:
scope: auto
code_generation:
docstrings:
language: en-US
path_instructions: []
unit_tests:
path_instructions: []
|
a38ae1e
to
622eaeb
Compare
622eaeb
to
d5bbd27
Compare
bcaaa35
to
e4774fe
Compare
7570271
to
de93dd1
Compare
04e222e
to
c62d9d4
Compare
d805c7d
to
ca568a5
Compare
e5f008f
to
95fdc65
Compare
95fdc65
to
c3930b2
Compare
🤖 I have created a release beep boop
1.5.0 (2025-09-09)
Features
Bug Fixes
This PR was generated with Release Please. See documentation.