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

Less merge conflicts *hurray* :) #609

Merged
merged 3 commits into from
Apr 7, 2018

Conversation

tralafiti
Copy link
Contributor

Moved compiled css into a version tag folder - analog to structure of build js files. This should help decrease the likelihood of conflicts between forks and allows for a quick switch between different builds of the frondend code.

Would be greatly appreciated if you could integrated this :-)

- remove base-url from inline-images. path is resolved by compass (images_dir)
- commend out compass configs that get overwritten in gulpfile
- update gulpfile to build into version tag folder
- update paths in css and templates accordingly
- fix version tag placement in compass options
# Conflicts:
#	public/css/pathfinder.css
#	public/css/v1.3.4/pathfinder.css.map
@exodus4d
Copy link
Owner

exodus4d commented Apr 7, 2018

Thanks for this! Code looks clean and well documented :)
Glad to see, that JS and CSS build files, both have a version based dist folder now, I had this in my mind for a long time...
The old parameterized public/css/pathfinder.css?v1.3.4 url worked fine for cache busting but did not allow multiple CSS builds side by side.
Maybe it´s worth to add a task:cleanCssAll Gulp task in future that clears all outdated CSS version directories, if someone needs to. ..
I´m not a NodeJs/Gulp expert at all and happy if it does the job I want to, but switching to Gulp4 was one of the best decisions, -> lots of fun.

btw. feel free to join our Slack chat, it makes discussions easier. pathfinder@exodus4d.de

@exodus4d exodus4d merged commit 7992cf5 into exodus4d:develop Apr 7, 2018
@exodus4d exodus4d added this to the v1.3.4 milestone Apr 7, 2018
@exodus4d exodus4d self-assigned this Apr 7, 2018
@exodus4d exodus4d self-requested a review April 7, 2018 13:00
exodus4d added a commit that referenced this pull request Apr 7, 2018
@tralafiti tralafiti deleted the feature/groundwork branch April 8, 2018 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants