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

Lowercase filenames and add lang attribute to all specs #102

Merged
merged 4 commits into from
Jun 29, 2019

Conversation

sorpaas
Copy link
Contributor

@sorpaas sorpaas commented Jun 29, 2019

  • Lowercased spec filenames -- this fits better with the rest of folder structure and generated permalink is lowercased anyway.
  • Add lang item to all specs.

@sorpaas sorpaas requested a review from a team June 29, 2019 06:25
Copy link
Contributor

@YazzyYaz YazzyYaz left a comment

Choose a reason for hiding this comment

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

Lang stuff looks great.

What's the reason for lowercase filenames again?

@@ -13,20 +13,20 @@ <h2>ECIP Types</h2>

<p>ECIPs are separated into a number of types, and each has its own list of ECIPs.</p>

<h3>Standard Track ({{site.specs|where:"type","Standards Track"|size}})</h3>
<h3>Standard Track ({{site.specs|where:"lang","en"|where:"type","Standards Track"|size}})</h3>
Copy link
Contributor

Choose a reason for hiding this comment

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

Cool stuff. This will make it easier to add more languages in the future for all of them and explore each ECIP.

@sorpaas
Copy link
Contributor Author

sorpaas commented Jun 29, 2019

@YazzyYaz

What's the reason for lowercase filenames again?

It's nicer, and EIP repo uses lowercase as well. :)

Copy link
Contributor

@YazzyYaz YazzyYaz left a comment

Choose a reason for hiding this comment

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

LGTM

@sorpaas sorpaas merged commit b984ab6 into master Jun 29, 2019
@soc1c soc1c deleted the sp-lowercase-lang branch February 3, 2020 10:34
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.

3 participants