Skip to content

Rename AIBundle to AiBundle, etc. #217

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Jul 30, 2025
Merged

Rename AIBundle to AiBundle, etc. #217

merged 9 commits into from
Jul 30, 2025

Conversation

fabpot
Copy link
Member

@fabpot fabpot commented Jul 28, 2025

Q A
Bug fix? yes
New feature? no
Docs? no
Issues n/a
License MIT

Rename AIBundle to AiBundle to follow Symfony naming convention.

@OskarStark
Copy link
Contributor

recipe, would need an update as well

@OskarStark OskarStark mentioned this pull request Jul 28, 2025
Merged
@OskarStark OskarStark added the AI Bundle Issues & PRs about the AI integration bundle label Jul 28, 2025
@chr-hertel
Copy link
Member

Does that also apply to namespaces like OpenAI or GPT, which are a more based on the actual product or brand name?

@fabpot
Copy link
Member Author

fabpot commented Jul 28, 2025

Does that also apply to namespaces like OpenAI or GPT, which are a more based on the actual product or brand name?

Indeed, that should be OpenAI or Gpt, like we're using Postgresql and not PostgreSQL.

@fabpot
Copy link
Member Author

fabpot commented Jul 28, 2025

Renamed OpenAI and GPT.

welcoMattic
welcoMattic reviewed Jul 28, 2025
View reviewed changes
@fabpot
Copy link
Member Author

fabpot commented Jul 28, 2025

Back to draft as there is more :)

@fabpot fabpot marked this pull request as draft July 28, 2025 09:23
welcoMattic
welcoMattic reviewed Jul 28, 2025
View reviewed changes
@fabpot fabpot force-pushed the case-fix branch 3 times, most recently from 1d37634 to 492b0a6 Compare July 28, 2025 09:30
welcoMattic
welcoMattic reviewed Jul 28, 2025
View reviewed changes
@chr-hertel chr-hertel added Platform Issues & PRs about the AI Platform component Agent Issues & PRs about the AI Agent component Store Issues & PRs about the AI Store component Demo Issues & PRs about the demo application labels Jul 29, 2025
@fabpot fabpot force-pushed the case-fix branch 3 times, most recently from c6967ba to 529f3fd Compare July 30, 2025 09:01
@fabpot fabpot marked this pull request as ready for review July 30, 2025 09:09
@fabpot
Copy link
Member Author

fabpot commented Jul 30, 2025

This one should be ready now @chr-hertel

@nicolas-grekas nicolas-grekas changed the title Rename AIBundle to AiBundle Rename AIBundle to AiBundle, etc. Jul 30, 2025
@OskarStark
Copy link
Contributor

Love the etc. suffix in the PR title 😄

@fabpot fabpot merged commit 1da4f58 into symfony:main Jul 30, 2025
12 of 13 checks passed
@chr-hertel chr-hertel mentioned this pull request Jul 31, 2025
Merged
chr-hertel added a commit that referenced this pull request Aug 1, 2025
@chr-hertel
bug #238 [AI Bundle] Fix profiler template reference (chr-hertel)
90a5d10 
This PR was merged into the main branch.

Discussion
----------

[AI Bundle] Fix profiler template reference

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        |
| License       | MIT

Related to #217

Commits
-------

122747c Fix profiler template reference
@chr-hertel chr-hertel mentioned this pull request Aug 1, 2025
Merged
OskarStark added a commit that referenced this pull request Aug 1, 2025
@OskarStark
bug #241 [Examples] Fix OpenAI and GPT namespaces in Neo4j RAG exampl…
f924bed 
…e (chr-hertel)

This PR was merged into the main branch.

Discussion
----------

[Examples] Fix OpenAI and GPT namespaces in Neo4j RAG example

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        |
| License       | MIT

Follow up after #217 and #183

Commits
-------

7770885 Fix OpenAI and GPT namespaces in Neo4j RAG example
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@welcoMattic welcoMattic welcoMattic left review comments

@OskarStark OskarStark OskarStark approved these changes

@derrabus derrabus derrabus approved these changes

@chr-hertel chr-hertel chr-hertel approved these changes

@Nyholm Nyholm Awaiting requested review from Nyholm Nyholm is a code owner

Assignees
No one assigned
Labels
Agent Issues & PRs about the AI Agent component AI Bundle Issues & PRs about the AI integration bundle Demo Issues & PRs about the demo application Platform Issues & PRs about the AI Platform component Store Issues & PRs about the AI Store component
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@fabpot @OskarStark @chr-hertel @welcoMattic @derrabus