Skip to content

chore: remove stale TODOs and dead code#434

Open
sanjana2505006 wants to merge 2 commits intoapache:masterfrom
sanjana2505006:chore/remove-stale-todos
Open

chore: remove stale TODOs and dead code#434
sanjana2505006 wants to merge 2 commits intoapache:masterfrom
sanjana2505006:chore/remove-stale-todos

Conversation

@sanjana2505006
Copy link
Copy Markdown
Contributor

Removes large sections of commented-out HTML and outdated TODO metadata from community.html to improve source cleanliness.

@sanjana2505006
Copy link
Copy Markdown
Contributor Author

Hello @jamesfredley, While reviewing pages/community.html, I saw that there were large chunks of commented-out HTML code.
This PR seeks to remove these commented-out chunks to make the source more readable. Specifically:
The old Conferences section, which was commented out with a TODO about 2020 events.
A few broken Meetup links in the User Groups section, which were commented out with a TODO about broken links.
There are no changes to functionality; this is purely a cleanup to make the source file more readable.

Copy link
Copy Markdown
Contributor

@sbglasius sbglasius left a comment

Choose a reason for hiding this comment

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

@jamesfredley we should discuss if this page is even relevant any more. The only active group is that in Madrid.

Comment thread pages/community.html
Comment on lines 137 to 139
<li>
<a href="https://www.meetup.com/grails/">United States - NYC Groovy / Grails Meetup</a>
</li>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is also not there any more 🙁

Comment thread pages/community.html
Comment on lines 141 to 144
<li>
<a href="https://www.meetup.com/java-161/">United States - SF Bay Groovy and Grails Meetup
Group</a>
</li>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No activity since 2015 🙁

@jamesfredley should we remove?

Comment thread pages/community.html
Comment on lines 145 to 148
<li>
<a href="https://www.meetup.com/st-louis-groovy-and-grails-meetup/">United States - St. Louis
Groovy &amp; Grails Meetup</a>
</li>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is also not there any more 🙁

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