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

(bug) (types) Workaround for mas account failing on macOS >= 12... now also include macOS 13 #48

Merged
merged 2 commits into from
Jan 24, 2023

Conversation

lucymhdavies
Copy link
Contributor

Addition to #43

Very obviously super hacky... but it's a quick and dirty GOOD ENOUGH for now...

We experimented with Greater Than Or Equal To stuff in the original PR, but never got it working.

May experiment further at some point... especially as mas-cli/mas#417 appears to be going nowhere fast

@github-actions github-actions bot added types Relates to types and assertions, i.e. not the core Bork code bug Something isn't working labels Jan 24, 2023
github-actions[bot]
github-actions bot previously approved these changes Jan 24, 2023
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

1 Warning
⚠️ Are you sure we don't need to add/update tests?

Generated by 🚫 Danger

@github-actions
Copy link

1 Warning
⚠️ This PR does not refer to an existing milestone

Generated by 🚫 Danger

…w also include macOS 13

Addition to borksh#43

Very obviously super hacky... but it's a quick and dirty GOOD ENOUGH for now...
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

✅ Good on 'ya.
Are you sure we don't need to add/update tests?

Generated by 🚫 Danger

@skylarmacdonald
Copy link
Member

Hah. I'll take it. Let me fix the Ubuntu test workflow and then I'll approve now that Danger is happy.

@skylarmacdonald
Copy link
Member

skylarmacdonald commented Jan 24, 2023

@lucymhdavies could I get you to merge main into this and push again please? just to force it to re-run the tests now I've fixed the Linux one.
Edit: Disregard — didn't think I could do it, but I can.
Edit 2: … but you've done it now anyway so all is well!

@lucymhdavies
Copy link
Contributor Author

Sure thing

Edit: neat

Edit 2: 2 fast 4 u ;)

@skylarmacdonald skylarmacdonald added this to the v0.14.x milestone Jan 24, 2023
@skylarmacdonald skylarmacdonald merged commit 9d7561a into borksh:main Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working types Relates to types and assertions, i.e. not the core Bork code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants