Skip to content

Setup new module from specifications#1

Merged
maxime-rainville merged 2 commits into
masterfrom
cursor/setup-new-module-from-specifications-ae5d
Oct 17, 2025
Merged

Setup new module from specifications#1
maxime-rainville merged 2 commits into
masterfrom
cursor/setup-new-module-from-specifications-ae5d

Conversation

@maxime-rainville

@maxime-rainville maxime-rainville commented Oct 13, 2025

Copy link
Copy Markdown
Member

This PR is the initial implementation of our new "well known" module for Silverstripe.

The broad purpose is to make it easy to define entries under .well-known.


Open in Cursor Open in Web

@cursor

cursor Bot commented Oct 13, 2025

Copy link
Copy Markdown

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@maxime-rainville maxime-rainville force-pushed the cursor/setup-new-module-from-specifications-ae5d branch from 814adf7 to 145aad7 Compare October 14, 2025 04:28
Comment thread .github/workflows/ci.yml

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

That's just basic CI stuff. We're running some unit test, linting and PHPStan.

@maxime-rainville maxime-rainville marked this pull request as ready for review October 14, 2025 04:33

@elliotdobson elliotdobson left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Clean implementation!

@maxime-rainville maxime-rainville merged commit f27039c into master Oct 17, 2025
5 checks passed
@maxime-rainville maxime-rainville deleted the cursor/setup-new-module-from-specifications-ae5d branch October 17, 2025 00:28
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.

4 participants