Skip to content

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

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 Jun 24, 2025

This PR adds new changelog entries.


Important

Adds a new changelog entry for "Generate OpenAPI specification from HTTP routes and webhooks" to changelogs.ts.

  • Changelog Update:
    • Adds a new entry for "Generate OpenAPI specification from HTTP routes and webhooks" to changelogs.ts with a date of '2025-06-18'.

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

Copy link

cloudflare-workers-and-pages bot commented Jun 24, 2025

Deploying windmill with  Cloudflare Pages  Cloudflare Pages

Latest commit: 21d7bbe
Status: ✅  Deploy successful!
Preview URL: https://e67bb64e.windmill.pages.dev
Branch Preview URL: https://changelog-update-15846791094.windmill.pages.dev

View logs

@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 21d7bbe in 32 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:9
  • Draft comment:
    New changelog entry added. Verify that the date '2025-06-18' is intentional and maintains proper descending order with other entries.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_mTzInMHjFyiNb2Un

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 Jun 24, 2025

Claude finished @rubenfiszel's task —— View job


PR Review Complete ✅

Todo List:

  • Read the changed file to understand modifications
  • Analyze changelog entry format and content
  • Provide comprehensive review feedback

Review Summary

Changes Made:

  • Added a new changelog entry for "Generate OpenAPI specification from HTTP routes and webhooks" in frontend/src/lib/components/sidebar/changelogs.ts:8-12
  • Entry includes proper label, href, and date fields following existing format

Code Quality Assessment

Format Consistency: The new entry perfectly matches the existing changelog entry format with all required fields (label, href, date)

Date Format: Uses the correct YYYY-MM-DD format consistent with all other entries

Chronological Ordering: Maintains proper reverse chronological order (newest entries first):

  • New entry: 2025-06-18
  • Next entry: 2025-03-11 (MQTT triggers)

URL Pattern: The href follows the established pattern https://www.windmill.dev/changelog/[feature-slug]

TypeScript Compliance: No type errors, proper object structure matching the Changelog interface

Code Formatting: Consistent indentation and spacing with the rest of the file

Observations

📅 Future Date: The entry date 2025-06-18 appears to be in the future (assuming today's date is before this). This may be intentional if the changelog entry is scheduled for a future release.

Verdict

LGTM 👍 This is a clean, properly formatted changelog addition that follows all existing conventions. The change is minimal and low-risk, adding valuable information about a new OpenAPI generation feature.


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