Skip to content

docs: add field-level ignore syntax to .kbcignore documentation#371

Merged
Matovidlo merged 2 commits intomainfrom
devin/1773215717-kbcignore-field-level-docs
Mar 11, 2026
Merged

docs: add field-level ignore syntax to .kbcignore documentation#371
Matovidlo merged 2 commits intomainfrom
devin/1773215717-kbcignore-field-level-docs

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot commented Mar 11, 2026

Jira issue(s): Related to PSGO-201 (implementation in keboola/keboola-as-code#2547)

Changes:

  • Extends the .kbcignore documentation to cover the new field-level ignore syntax (componentID/configID:fieldName) introduced in keboola-as-code
  • Restructures the existing .kbcignore section with subheadings for clarity: "Configuration and Row Ignore", "Field-Level Ignore", and "Configuration-Level Ignore"
  • Documents bidirectional behavior (push keeps remote value, pull keeps local value)
  • Adds examples for isDisabled struct field and schedule.cronTab dot-notation content key
  • Notes validation rules (no empty field name, no leading/trailing dots)

Review checklist

  • Heading hierarchy: Three new ### subheadings were added. Verify the "Configuration-Level Ignore" heading before the existing push/pull operation descriptions reads naturally.
  • Behavioral accuracy: The bidirectional semantics (push discards local, pull discards remote for the ignored field) should be verified against the implementation in keboola-as-code#2547.
  • Struct-level field list: Only isDisabled is documented — confirm no other struct fields are currently supported.
  • Rendered output: Jekyll build passes, but visual rendering should be checked (alert box, heading anchors, code blocks).

Link to Devin session: https://app.devin.ai/sessions/db61451f2e3c485bb465d5bb52b21048
Requested by: Martin Vasko (martin.vasko@keboola.com)

Co-Authored-By: Martin Vasko <Matovidlo2@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

devin-ai-integration bot commented Mar 11, 2026

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@linear
Copy link
Copy Markdown

linear bot commented Mar 11, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the CLI documentation for .kbcignore to describe the newly supported field-level ignore syntax, including examples and push/pull semantics, and reorganizes the .kbcignore section for readability.

Changes:

  • Expands .kbcignore docs to include field-level ignore rules (component/config:field), with examples.
  • Adds subheadings to separate configuration/row ignores vs field-level ignores vs behavior details.
  • Documents bidirectional behavior (push keeps remote for ignored field; pull keeps local) and basic validation rules.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-Authored-By: Martin Vasko <Matovidlo2@gmail.com>
Copy link
Copy Markdown
Contributor

@hosekpeter hosekpeter left a comment

Choose a reason for hiding this comment

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

👍

@Matovidlo Matovidlo merged commit bf907b0 into main Mar 11, 2026
1 of 2 checks passed
@Matovidlo Matovidlo deleted the devin/1773215717-kbcignore-field-level-docs branch March 11, 2026 09:18
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.

3 participants