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

Refactor gulpfile.js #137

Merged
merged 8 commits into from
Aug 23, 2022
Merged

Refactor gulpfile.js #137

merged 8 commits into from
Aug 23, 2022

Conversation

Raruto
Copy link
Collaborator

@Raruto Raruto commented Aug 12, 2022

a partial refactor is actually needed (or at least suggested) before complete #68

based on #35


Notable changes:

  • remove function interpolateVersion()
  • remove function set_current_hash_version()
  • remove task g3w-admin:client_only
  • remove task set_build_all_to_false
  • remove variable g3w_admin_version

@Raruto Raruto added config Anything related to configuration files refactoring Anything which could result in a API change labels Aug 12, 2022
gulpfile.js Show resolved Hide resolved
gulpfile.js Show resolved Hide resolved
@Raruto Raruto added this to the v3.7 milestone Aug 12, 2022
@Raruto Raruto changed the title Gulpfile refactor Refactor gulpfile.js Aug 12, 2022
# Conflicts:
#	gulpfile.js
@Raruto
Copy link
Collaborator Author

Raruto commented Aug 23, 2022

@volterra79 if that's okay with you I'd say we can quite safely merge this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config Anything related to configuration files refactoring Anything which could result in a API change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants