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

fix(create-local-vue): fix max range stack size (fix #1768) #1786

Merged
merged 2 commits into from Feb 11, 2021

Conversation

antoinerey
Copy link
Contributor

@antoinerey antoinerey commented Feb 10, 2021

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

The PR fulfills these requirements:

Other information:

@antoinerey
Copy link
Contributor Author

While this PR is expected to fail, it seems that every checks are green which is quite confusing. 🤔 😁

const sets = Array.from({ length: 2 })

it.each(sets)(
// Note, the message may not be the final one, based on what I discover.
Copy link
Member

Choose a reason for hiding this comment

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

can you put link to the issue? it will be difficult to remember what this test is really doing

@antoinerey antoinerey marked this pull request as ready for review February 11, 2021 15:42
@antoinerey antoinerey changed the title test(create-local-vue): fix max range stack size (fix #1768) fix(create-local-vue): fix max range stack size (fix #1768) Feb 11, 2021
@lmiller1990
Copy link
Member

Let's hope this does not break anything 👀

@lmiller1990 lmiller1990 merged commit 30e5bc3 into vuejs:dev Feb 11, 2021
@codfish
Copy link

codfish commented Apr 6, 2021

@lmiller1990 what's the release schedule for the project like? When do you think we'll see this fix released into the wild?

@lmiller1990
Copy link
Member

lmiller1990 commented Apr 7, 2021

There is no set schedule, it's just whenever I do one. I can do a release this week. Maybe we should investigate sem-release and automate things...

If no release within a few days, ping me again, but I should be able to do it soon.

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.

None yet

3 participants