Skip to content
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

Prepare 7.2 branch #19915

Merged
merged 1 commit into from
Jun 17, 2024
Merged

Conversation

alamirault
Copy link
Contributor

No description provided.

@carsonbot carsonbot added this to the 7.2 milestone May 24, 2024
@@ -48,10 +48,10 @@ application:
.. code-block:: terminal

# run this if you are building a traditional web application
$ symfony new my_project_directory --version="7.1.*" --webapp
$ symfony new my_project_directory --version="7.2.*" --webapp
Copy link
Member

Choose a reason for hiding this comment

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

I tried this command and it doesn't work:

❯ symfony new my_project_directory --version="7.2.*" --webapp

In CreateProjectCommand.php line 424:

  Could not find package symfony/skeleton with version 7.2.*.

These other two commands worked as expected:

❯ symfony new my_project_directory --version="7.2.x-dev" --webapp
  Creating a new Symfony 7.2.x-dev project with Composer
  [...]

❯ symfony new my_project_directory --version=next --webapp
  Creating a new Symfony 7.2.x@dev project with Composer
  [...]

Copy link
Member

Choose a reason for hiding this comment

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

Would you agree on using --version=next here? That way, we wouldn't have to update this in the dev branch of Symfony Docs ... but we'd need to update it back to an specific constraint version whenever a new minor Symfony version is released.

Copy link
Member

Choose a reason for hiding this comment

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

I like this idea. 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

Me too

Copy link
Member

Choose a reason for hiding this comment

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

At the end I used 7.2.x-dev because we need to use that in Composer commands (next doesn't work there). This way, we use 7.2.x-dev everywhere and this will be easier to maintain in the future.

@javiereguiluz javiereguiluz merged commit 92dff5f into symfony:7.2 Jun 17, 2024
3 checks passed
@javiereguiluz
Copy link
Member

Thanks Antoine!

@alamirault alamirault deleted the feature/prepare-72-branch branch June 18, 2024 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants