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 annoying answers version bug #1007

Merged
merged 1 commit into from
Oct 23, 2024
Merged

Conversation

xexyl
Copy link
Contributor

@xexyl xexyl commented Oct 23, 2024

Now the answers version can go in soup/version.h and the EOF marker can go in mkiocccentry.h

At the same time the test_ioccc/prep.sh had a bug fix that prevents the line of the form (regex without quotes) '^OK$' from being let in the log file.

As these were annoying bugs, especially the answers version one, it seemed worth making another pull request, but this is likely my last one until the Great Fork Merge in the other repo: though I guess we'll have to wait and 'C'. :-)

Now the answers version can go in soup/version.h and the EOF marker can
go in mkiocccentry.h

At the same time the test_ioccc/prep.sh had a bug fix that prevents the
line of the form (regex without quotes) '^OK$' from being let in the log
file.

As these were annoying bugs, especially the answers version one, it
seemed worth making another pull request, but this is likely my last one
until the Great Fork Merge in the other repo: though I guess we'll have
to wait and 'C'. :-)
@xexyl
Copy link
Contributor Author

xexyl commented Oct 23, 2024

Aka a while .. when back I will work on the FAQ 'over there'. This seemed useful enough to make a pull request. Expect FAQ updates over there in a while!

Copy link
Contributor

@lcn2 lcn2 left a comment

Choose a reason for hiding this comment

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

Good idea 👍, @xexyl

@lcn2 lcn2 merged commit 69097cc into ioccc-src:master Oct 23, 2024
3 checks passed
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