Skip to content

Added add_fields workflow - #18548

Merged
melinath merged 2 commits into
GoogleCloudPlatform:mainfrom
melinath:field-workflow
Aug 7, 2026
Merged

Added add_fields workflow#18548
melinath merged 2 commits into
GoogleCloudPlatform:mainfrom
melinath:field-workflow

Conversation

@melinath

@melinath melinath commented Aug 6, 2026

Copy link
Copy Markdown
Member

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.


@modular-magician

Copy link
Copy Markdown
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 989fab9:

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

@melinath
melinath requested a review from c2thorn August 6, 2026 17:56

@c2thorn c2thorn 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.

Looks great. Approving now for whenever GHA comes back online.


Add the requested field(s) to the requested resource(s).

### 3. Add tests for the fields

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.

Very strong, I like this section a lot. New resource should probably route to this instead of default-workflow https://github.com/GoogleCloudPlatform/magic-modules/blob/main/.agents/skills/workflows/new_resource/SKILL.md#2b-manual-path

Outside of this PR's scope, just noting.

@melinath
melinath enabled auto-merge August 6, 2026 20:08
@slevenick

Copy link
Copy Markdown
Contributor

/gcbrun

@modular-magician

Copy link
Copy Markdown
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 989fab9:

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

@slevenick

Copy link
Copy Markdown
Contributor

/gcbrun

@modular-magician

Copy link
Copy Markdown
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 989fab9:

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

@modular-magician

Copy link
Copy Markdown
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 3e746b8:

Diff report

Your PR hasn't generated any diffs, but I'll let you know if a future commit does.

@melinath
melinath added this pull request to the merge queue Aug 7, 2026
Merged via the queue into GoogleCloudPlatform:main with commit 67c69ea Aug 7, 2026
25 of 27 checks passed
@melinath
melinath deleted the field-workflow branch August 7, 2026 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants