Skip to content
This repository has been archived by the owner on Sep 16, 2019. It is now read-only.

Removed duplicate what-input.js path #640

Closed
wants to merge 2 commits into from
Closed

Removed duplicate what-input.js path #640

wants to merge 2 commits into from

Conversation

EricRihlmann
Copy link
Contributor

The gulpfile.js had a duplicate reference to assets/components/what-input/what-input.js. I removed the first reference and kept the second one with the comment above it.

@colin-marshall
Copy link
Collaborator

Thanks @EricRihlmann. This was my fault. If you don't mind, would you actually remove the 2nd reference to what-input instead of the first please? I'm trying to keep it the same as: https://github.com/zurb/foundation-zurb-template.

Also, please fix the build errors so it can be merged. Thanks!

@EricRihlmann
Copy link
Contributor Author

@colin-marshall no worries, I just pushed a commit with your suggestion.

About the build errors, they appear to be completely unrelated to my changes. The errors are with improper case for two PHP variables. How did you get your builds to pass?

FILE: ...dpress/src/wp-content/themes/FoundationPress/library/cleanup.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 247 | ERROR | Variable "Foundationpress_img_rebuilder" is not in valid
     |       | snake_case format
     |       | (WordPress.NamingConventions.ValidVariableName.NotSnakeCase)
----------------------------------------------------------------------
FILE: ...hemes/FoundationPress/library/protocol-relative-theme-assets.php
----------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
----------------------------------------------------------------------
 90 | ERROR | Variable "Foundationpress_protocol_relative_theme_assets"
    |       | is not in valid snake_case format
    |       | (WordPress.NamingConventions.ValidVariableName.NotSnakeCase)
----------------------------------------------------------------------

@colin-marshall
Copy link
Collaborator

@EricRihlmann you can copy the changes from my PR to fix those build errors:
https://github.com/olefredrik/FoundationPress/pull/644/files

@EricRihlmann
Copy link
Contributor Author

Shouldn't I just ignore the build errors since my pull request is unrelated to them? Plus you've already issued a PR with a fix.

@colin-marshall
Copy link
Collaborator

Can a PR still be merged if it has a build failure? If so, then yes you are correct and you can leave it as is. I'm a noob when it comes to the build stuff.

@olefredrik
Copy link
Owner

Fixed in #656

@olefredrik olefredrik closed this Dec 27, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants