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

Add python pre-req and quotes for mac curl #776

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

SteveLasker
Copy link
Contributor

@SteveLasker SteveLasker commented Feb 23, 2024

@beekman33 noticed the url for the events API didn't work on a mac. Enclosing with quotes worked.
We also added a generic link to the python dependency.

Signed-off-by: steve lasker <stevenlasker@hotmail.com>
@pcnorth
Copy link
Contributor

pcnorth commented Feb 26, 2024

I noticed that some of the other API reference pages have a mixture of curl examples with quotes and others without quotes. Both work on git bash so I'm thinking that we should make it standard to use quotes.

Does that sound like a good idea?

@pcnorth
Copy link
Contributor

pcnorth commented Feb 26, 2024

One more thing, the other curl examples in the page do not have quotes. Do they all work on a MAC?

@SteveLasker
Copy link
Contributor Author

+1 for consistency.
who’s got a mac that can run through them?

@pcnorth
Copy link
Contributor

pcnorth commented Feb 26, 2024

I know that Madia and Rusty have MACs. There might be others, possibly John Hartley.

@SteveLasker
Copy link
Contributor Author

I have an old Mac I can try and revive going forward. For the sake of expediency, can we merge this one as we know this fixes the current issues @beekman33 hit, and make additional consistency fixes as a separate PR?

Copy link
Contributor

@pcnorth pcnorth left a comment

Choose a reason for hiding this comment

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

Approved

@SteveLasker SteveLasker merged commit 30c1300 into main Feb 28, 2024
1 check passed
@SteveLasker SteveLasker deleted the steve/scitt-mac-updates branch July 18, 2024 20:43
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

2 participants