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

Cleaning up document structure and duplicate entries #335

Merged
merged 5 commits into from
Dec 5, 2023

Conversation

galeep
Copy link
Contributor

@galeep galeep commented Dec 4, 2023

As mentioned in #329: this first change in a series aims to enhance the maintainability, consistency, and overall usability of the list. I'm submitting these incrementally to manage the changeset size and minimize drift.

This PR addresses:

  1. Heading Consistency: I rectified inconsistencies in about 18 out of 300+ entries by changing the heading level from H5 to H4. This improves readability and parsability.
  2. Duplicate Resolution: I merged duplicate country and person entries in the same section. These changes focus on structural consistency in the document. Any entries contributed under different country categories will remain as they are for now.
  3. ToC Updates: I have regenerated the table of contents based on the regions specified in the document.

These changes primarily target elements causing parsing difficulties with semi-structured data.

Next steps:

Alphabetization by last name: the upcoming PRs will address alphabetization issues, as per #329
Automation: after addressing alphabetizing, I'll provide the scripts to keep listings alphabetized and the ToC accurate and up-to-date.

P.S. On a technical note, I've introduced a marker (<!-- toc -->) for automated ToC generation. This change will not disrupt any existing ToC generation using the marker that was already present.

@galeep galeep marked this pull request as draft December 4, 2023 03:23
@galeep galeep marked this pull request as ready for review December 4, 2023 03:30
@galeep
Copy link
Contributor Author

galeep commented Dec 4, 2023

Cleaned up an orphaned H4 entry for Japan in Organizers section, leaving the associated organizer's entry in Asia.
(Based on the comment and change in #316, the entry may need to be updated to reflect current info.)

Copy link
Collaborator

@jlooper jlooper 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!

@jlooper jlooper merged commit aa0509f into fempire:main Dec 5, 2023
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