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

Support macOS 13 Ventura #80

Closed
wants to merge 1 commit into from
Closed

Support macOS 13 Ventura #80

wants to merge 1 commit into from

Conversation

devinea
Copy link

@devinea devinea commented Nov 3, 2022

Would be great if this can be merged and a new version of insight released.

@AlexSwensen
Copy link

Sadly its a little more cumbersome than just that.

See here for details.

If you are having issues trying to run yo in Ventura, you can follow these workaround instructions.

@devinea
Copy link
Author

devinea commented Nov 3, 2022

@AlexSwensen Great that another fix in progress. I'll close this.

@devinea devinea closed this Nov 3, 2022
@mischah
Copy link
Member

mischah commented Nov 7, 2022

Released a fix for Yeoman with Yeoman v4.3.1 :octocat:

@rprieto
Copy link
Contributor

rprieto commented Jan 4, 2023

Hi @AlexSwensen. Is there a plan to support macOS Ventura?
It seems that the fix above was to temporarily remove insights from yo, is that right?
I'm asking because this also impacts any other tool using insights.

Thank you

@AlexSwensen
Copy link

Hi @AlexSwensen. Is there a plan to support macOS Ventura?

It seems that the fix above was to temporarily remove insights from yo, is that right?

I'm asking because this also impacts any other tool using insights.

Thank you

From what I understand, that is correct. I would go to the main yo repo to ask for more details on this. I left a link in my original comment above.

@rprieto
Copy link
Contributor

rprieto commented Jan 4, 2023

Thanks. I don’t actually use yo itself. This is about Thumbsup that uses insights directly for analytics, and which doesn’t build on macOS Ventura anymore.

thumbsup/thumbsup#322

@AlexSwensen
Copy link

@rprieto In that case I would either open a new issue, or fork the package and patch it. The workarounds in those same links should work for you.

@rprieto
Copy link
Contributor

rprieto commented Jan 10, 2023

Thanks @AlexSwensen, I will do that. (sorry I just realised you're not a member of the project)

@mischah
Copy link
Member

mischah commented Jan 10, 2023

Opened a new issue to address this which mentions the work that is to do fix this.
See #83

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

4 participants