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

Output exact Cigar JSON parse error #46

Merged
merged 1 commit into from
Sep 19, 2023
Merged

Conversation

mbrodala
Copy link
Contributor

Also avoid the term ".cigar.json" which is misleading if a custom file is used.

@Brunty
Copy link
Owner

Brunty commented Jul 14, 2022

Cool, thanks for this!

I'll check out the build failures when I get chance this weekend, they're not something you've caused - the whole thing has issues it seems (I haven't touched this project in a while)

@Brunty
Copy link
Owner

Brunty commented Jul 16, 2022

@mbrodala I've fixed the test issues so if you pull in Brunty:master to your branch it should be all good :)

Also avoid the term ".cigar.json" which is misleading if a custom file is used.
@mbrodala
Copy link
Contributor Author

Thanks, did that now. All green. 💚

@mbrodala
Copy link
Contributor Author

@Brunty Is there something left to do here aside from merge?

@mbrodala
Copy link
Contributor Author

@Brunty Any update?

@Brunty Brunty merged commit 8d5d8fa into Brunty:master Sep 19, 2023
@Brunty
Copy link
Owner

Brunty commented Sep 19, 2023

Hey sorry for the delay! I'll get a new release tagged shortly :)

Thanks for the work!

Brunty pushed a commit that referenced this pull request Sep 19, 2023
Also avoid the term ".cigar.json" which is misleading if a custom file is used.
@mbrodala mbrodala deleted the patch-1 branch September 20, 2023 07:25
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.

2 participants