Skip to content

Change slug #866

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

Merged
merged 7 commits into from
Apr 3, 2025
Merged

Change slug #866

merged 7 commits into from
Apr 3, 2025

Conversation

colinleach
Copy link
Contributor

@colinleach colinleach commented Apr 3, 2025

added "old" to Annalyn's Infiltration

@colinleach colinleach requested a review from depial April 3, 2025 16:02
Copy link
Contributor

@depial depial left a comment

Choose a reason for hiding this comment

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

Let's hope this works

@depial depial merged commit 7f7d5b0 into exercism:main Apr 3, 2025
12 checks passed
@colinleach
Copy link
Contributor Author

Not looking great so far

@colinleach colinleach deleted the chenge-slug branch April 3, 2025 16:17
@depial
Copy link
Contributor

depial commented Apr 3, 2025

No, not good. The stub must be cached somewhere on the site, so maybe we need to wait for an update?

@colinleach
Copy link
Contributor Author

colinleach commented Apr 3, 2025

If nothing improves on the website in the next few hours, what's left?

  • Maybe remove concept.wip from eachexercise.jl (I'd vote for this)
  • The nuclear option: delete the old-annalyns-infiltration directory completely
  • Decide it's above our pay grade and focus on new content while somebody else fixes the legacy stuff

@depial
Copy link
Contributor

depial commented Apr 3, 2025

I'm just not sure how the site is interacting with that concept.wip directory, especially since it seems like it has it's own top-level config.json. I wish we could just have a clean slate, but the site seems very stubborn.

We could try removing concept.wip from eachexercise.jl, but I'm not sure that's going to change anything on the site.

I'm afraid of deleting the old-annalyns-infiltration directory, mainly because I'm not sure how permanent these changes which we tried to revert are going to be on the site.

Let's wait a bit. Remove the concept.wip from eachexercise.jl at some point, then see which of the other two options we want to go with. Sound good?

@colinleach
Copy link
Contributor Author

One (small) bit of good news. In the Annalyn's Infiltration online editor, I did a revert to start. That removed my old entry in game.jl, and on refresh I was given an annalyns-infiltration.jl stub. Added the solution, all tests passed.

I'm curious what a non-maintainer would see. Probably nothing, at this wip stage, but beyond that probably less visual noise.

@depial
Copy link
Contributor

depial commented Apr 3, 2025

I was able to reproduce that. I wonder if that would have gone through earlier with just the UUID swap?

I also see that before the slug name change, there were two exercises listed, but now it lists 1 exercise under Booleans (although there are still two links to the exercise)

image

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