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

Ignore main.js and head.js in github repo? #44

Open
benrito opened this issue Oct 5, 2018 · 2 comments
Open

Ignore main.js and head.js in github repo? #44

benrito opened this issue Oct 5, 2018 · 2 comments

Comments

@benrito
Copy link
Contributor

benrito commented Oct 5, 2018

Hey @tbeck, I've been working manually on the repo and notice that main.js and head.js want to update in Git whenever I'm working using browsersync. Should we add these to ignore list?

@tbeck
Copy link
Collaborator

tbeck commented Oct 5, 2018

Currently we aren't ignoring the bundled/transpiled versions of both of those files because they need to be available during the Jekyll build process. I haven't found a clean solution for bundling/transpiling JS in a Jekyll project.

We could definitely adjust the Gulp tasks so that new versions of head and main.js are only created when making changes to the JS source files or dependencies? That was our intention, but it seems like it is not working as expected.

I'm completely open to other ideas though. Let me know your thoughts?

@benrito benrito added this to the Future milestone Oct 6, 2018
@benrito
Copy link
Contributor Author

benrito commented Oct 6, 2018

Let's not worry about this now—will keep the ticket on ice in case we want to revisit.

@abandea abandea removed this from the Airbel 1.5 milestone Oct 7, 2019
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

No branches or pull requests

3 participants