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

Many fixes and other updates to guidelines #67

Merged
merged 2 commits into from
Jan 23, 2025

Conversation

xexyl
Copy link
Contributor

@xexyl xexyl commented Jan 23, 2025

This includes typos, fixing terms (according to the new terminology used by the IOCCC) and various other types of fixes.

Although I have gone through the entire file more does need to be done to make sure it all reads okay and that all links are correct.

This includes typos, fixing terms (according to the new terminology used
by the IOCCC) and various other types of fixes.

Although I have gone through the entire file more does need to be done
to make sure it all reads okay and that all links are correct.
@xexyl
Copy link
Contributor Author

xexyl commented Jan 23, 2025

Let's see if the issue of broken links in the script are reported here. I tested some that were reported broken on my end and they work fine on the server. What's more is that most of them I did not even touch ...

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.

Looks like some checks failed .. will wait until this issue is resolved.

@xexyl
Copy link
Contributor Author

xexyl commented Jan 23, 2025

I see the problem but if all is good it seems that there is a display problem with the script on my end (odd it did not show up here).

@xexyl
Copy link
Contributor Author

xexyl commented Jan 23, 2025

Looks like some checks failed .. will wait until this issue is resolved.

Fixed it seems. Missing '#'.

@xexyl
Copy link
Contributor Author

xexyl commented Jan 23, 2025

Unfortunately on my end I can't use the script as I see something really wrong: it shows all the links as bad. Not sure why that is. Very bizarre.

@xexyl
Copy link
Contributor Author

xexyl commented Jan 23, 2025

Kind of not a good sign. I see locally:

# Missing file linked in: guidelines.md
# Markdown line: FAQ on "[Mastodon](../faq.html#try_mastodon)"
./../faq.html

and every other link is broken too. Downloaded the file from here (the script) and same issue. But obviously that's not broken and everything works here ...

@xexyl
Copy link
Contributor Author

xexyl commented Jan 23, 2025

Kind of not a good sign. I see locally:

# Missing file linked in: guidelines.md
# Markdown line: FAQ on "[Mastodon](../faq.html#try_mastodon)"
./../faq.html

and every other link is broken too. Downloaded the file from here (the script) and same issue. But obviously that's not broken and everything works here ...

But the good thing at least is it's only this file. But why I can't figure out. Very odd.

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.

thank you @xexyl

@lcn2 lcn2 merged commit 38512e1 into ioccc-src:master Jan 23, 2025
1 check passed
@xexyl xexyl deleted the guidelines-updates branch January 24, 2025 17:55
@xexyl
Copy link
Contributor Author

xexyl commented Jan 24, 2025

thank you @xexyl

You are most welcome.

QUESTION for you @lcn2

Do you wish for the guidelines, rules and FAQ to refer to the simplified interface to mkiocccentry prior to it being done ?

@lcn2
Copy link
Contributor

lcn2 commented Jan 24, 2025

thank you @xexyl

You are most welcome.

QUESTION for you @lcn2

Do you wish for the guidelines, rules and FAQ to refer to the simplified interface to mkiocccentry prior to it being done ?

Sure.

@xexyl
Copy link
Contributor Author

xexyl commented Jan 24, 2025

thank you @xexyl

You are most welcome.

QUESTION for you @lcn2

Do you wish for the guidelines, rules and FAQ to refer to the simplified interface to mkiocccentry prior to it being done ?

Sure.

Okay I'll keep this in mind. I'm out of energy for now but hopefully tomorrow morning I can do more before company calls here.

Good luck with the server!

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