search

mgaudet / CompilerJobs

A listing of compiler, language and runtime teams for people looking for jobs in this area
https://mgaudet.github.io/CompilerJobs/
531 stars 64 forks source link

remove script tag from README #28

Closed kelj0 closed 4 years ago

mgaudet commented 4 years ago

This is issue #19

If you visit the rendered page: https://mgaudet.github.io/CompilerJobs/

You'll see that it's dynamically computing the count of companies; this is to avoid having a merge conflict every time a PR is opened.

Now: I'm not a Jekyll expert: It seems like it should be possible to create a "header" or a "footer" file that contains this script, and we could move it in there, so that only when README.md is compiled does it get the script.

I'd prefer we try to do that, than remove the dynamic update code entirely!

mgaudet commented 4 years ago

Sorry, I'm going to close this PR. You gave me the push I needed to finally figure out enough jekyll to setup a 'footer' file of sorts where this could get stored.