Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bower to npm - move angular 1 to npm #3344

Merged
merged 2 commits into from
Feb 13, 2018
Merged

Bower to npm - move angular 1 to npm #3344

merged 2 commits into from
Feb 13, 2018

Conversation

himdel
Copy link
Contributor

@himdel himdel commented Jan 30, 2018

Continuing the effort from #3343, overall description there.

This moves angular 1 from bower to npm, no other changes.

@ohadlevy
Copy link
Member

maybe I'm missing something obvious - but why not move it to webpack as well? I think long term it makes more sense to have npm packages + webpack vs npm + asset pipeline?

@miq-bot
Copy link
Member

miq-bot commented Feb 7, 2018

This pull request is not mergeable. Please rebase and repush.

@himdel
Copy link
Contributor Author

himdel commented Feb 7, 2018

Step by step :) You're right, it does .. but I want us to get off bower first, without dealing with moving stuff to webpack at the same time.

(See "Next steps" in #3343)

EDIT: updated the description accordingly

@miq-bot miq-bot changed the title [WIP] Bower to npm - move angular 1 to npm Bower to npm - move angular 1 to npm Feb 7, 2018
@miq-bot miq-bot removed the wip label Feb 7, 2018
"bootstrap-switch": "~3.3.4",
"moment": ">=2.10.5",
"bootstrap-select": "~1.12.2",
"font-awesome-sass": "~4.7.0"
"font-awesome-sass": "~4.7.0",
"angular": "1.6.9"
Copy link
Contributor Author

@himdel himdel Feb 7, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is autogenerated when running bower install -F, keeping for now, otherwise everybody will see a local change after bin/update.

@himdel himdel closed this Feb 8, 2018
@himdel himdel reopened this Feb 8, 2018
@himdel
Copy link
Contributor Author

himdel commented Feb 9, 2018

Test failure caused by Rails.application.config.assets.paths still containing bower_components, because railties thinks it's a good idea :).

Looking into whether we can remove it from the paths, or if we need to move bower elsewhere.


Looks like we can't, not in a rails engine. Moving bower to a different dir.

=> #3388

@himdel
Copy link
Contributor Author

himdel commented Feb 9, 2018

Pending #3388
@miq-bot add_label wip
@miq-bot remove_label wip

EDIT: merged

we're removing this, but better to add this for now than to deal with everybody adding this in their pr
@miq-bot
Copy link
Member

miq-bot commented Feb 12, 2018

Checked commits https://github.com/himdel/manageiq-ui-classic/compare/eeb27b0ed60088cd0dd81776af30bb9ca0ec8d8c~...1dc0bbd3427fb891cd361ee549aba34144b249ed with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. ⭐

@himdel
Copy link
Contributor Author

himdel commented Feb 13, 2018

Cc @mzazrivec ;)

@mzazrivec mzazrivec self-assigned this Feb 13, 2018
@mzazrivec mzazrivec added this to the Sprint 80 Ending Feb 26, 2018 milestone Feb 13, 2018
@mzazrivec mzazrivec merged commit eb34d83 into ManageIQ:master Feb 13, 2018
@himdel himdel deleted the bower-angualr1 branch February 14, 2018 15:41
@himdel
Copy link
Contributor Author

himdel commented Aug 3, 2018

issue: #3734

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants