-
Notifications
You must be signed in to change notification settings - Fork 43
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
Update SIGPLAN-AV page #250
Conversation
Hello, @janpaulpl! It would be really helpful if you could include some commentary about what changes this entails so I can navigate around them—any chance you could write some notes? Also, it looks like this includes a Python script to generate HTML. Did you know that this entire site is already generated HTML, including from data files with personnel listings like this? It uses Jekyll, and the data comes from various YAML files. For example, here's the listing of the executive committee: SIGPLAN.github.io/_data/Committees.yaml Lines 16 to 28 in 9bd36de
And then it gets formatted into HTML using Liquid templates; namely: SIGPLAN.github.io/ContactUs.md Line 11 in 9bd36de
https://github.com/SIGPLAN/SIGPLAN.github.io/blob/9bd36de7f4050a1a5b79f1fa191abaff7180edcd/_includes/committee-list.liquid Do you think we could use that existing functionality to format the HTML you're looking for, instead of creating an ad hoc approach just for this one list? |
* Members are now stored in YAML and rendered with templates * Update members as of last conference * Add pictures * Misc. restructuring and rewording of the text
No problem @sampsyo! We've removed all the ad-hoc HTML generation and replaced it with Liquid which is much cleaner. The PR also has a description of the changes now. |
Looks great; thanks!! |
AV page changes
make serve
command now has the--livereload
flag.About Us
section that explains what we provide SIGPLAN conferences in more detail.Added the following files:
_data/AVCommittee.yaml
_includes/av-committee.liquid
AV/index.md