Skip to content

docs(changelog): add new entries from changelog #6104

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

windmill-internal-app[bot]
Copy link
Contributor

@windmill-internal-app windmill-internal-app bot commented Jul 1, 2025

This PR adds new changelog entries.


Important

Adds a new changelog entry for 'Rust client SDK' to changelogs.ts.

  • Changelog Update:
    • Adds new entry for 'Rust client SDK' to changelogs.ts with date '2025-06-24'.

This description was created by Ellipsis for 04a8985. You can customize this summary. It will automatically update as commits are pushed.

@rubenfiszel
Copy link
Contributor

/ai review this PR

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 04a8985 in 35 seconds. Click for details.
  • Reviewed 17 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. frontend/src/lib/components/sidebar/changelogs.ts:8
  • Draft comment:
    New changelog entry for 'Rust client SDK' added. Verify that the URL and date (2025-06-24) maintain the descending order.
  • Reason this comment was not posted:
    Confidence changes required: 0% <= threshold 50% None

Workflow ID: wflow_oBMt1ewTIB6Qunrb

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

Copy link
Contributor

claude bot commented Jul 1, 2025

Claude finished @rubenfiszel's task —— View job


PR Review: Changelog Update

Todo List:

  • Read and analyze the changed file
  • Verify changelog entry format and structure
  • Check chronological ordering
  • Provide comprehensive review feedback

Review Summary

✅ APPROVED - The changelog entry follows the established format and is properly positioned.

Detailed Analysis

Format & Structure ✅

The new changelog entry follows the correct TypeScript structure:

{
    label: 'Rust client SDK',
    href: 'https://www.windmill.dev/changelog/rust-client-sdk',
    date: '2025-06-24'
}

Positives:

  • Proper TypeScript object structure with label, href, and date fields
  • Consistent formatting with trailing comma
  • Follows the established URL pattern https://www.windmill.dev/changelog/{slug}
  • Clean, descriptive label

Chronological Ordering ✅

The entry is correctly positioned at the top of the array, maintaining descending chronological order:

  • New entry: 2025-06-24 (Rust client SDK)
  • Previous first: 2025-03-11 (MQTT triggers)
  • This maintains the expected newest-first ordering

Consistency Check ✅

The entry is consistent with existing patterns:

  • URL slug matches the kebab-case convention
  • Date format follows ISO date format (YYYY-MM-DD)
  • Label is clear and descriptive
  • Proper spacing and indentation

Potential Considerations

  1. Date Validation: The date 2025-06-24 appears reasonable given the current date (2025-07-01)
  2. URL Accessibility: The changelog URL follows the established pattern and should resolve correctly
  3. Content Relevance: The "Rust client SDK" entry aligns with Windmill's multi-language support strategy

Recommendation

LGTM 🚀 - This is a clean, well-formatted changelog update that maintains consistency with the existing codebase structure and follows all established conventions.


Copy link

Deploying windmill with  Cloudflare Pages  Cloudflare Pages

Latest commit: 04a8985
Status: ✅  Deploy successful!
Preview URL: https://4ee2ec1d.windmill.pages.dev
Branch Preview URL: https://changelog-update-15999043351.windmill.pages.dev

View logs

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.

1 participant