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

445/bower-fix-and-cleanup #446

Merged
merged 5 commits into from
Oct 2, 2023
Merged

445/bower-fix-and-cleanup #446

merged 5 commits into from
Oct 2, 2023

Conversation

oscgonfer
Copy link
Contributor

@oscgonfer oscgonfer commented Jul 31, 2023

This is a PR to test changes on node/bower build process. The issue comes from the fact that bower installs need certs as explained here, but are no longer accepted because of old versions.

This PR changes a few things:

  • removes legacy packages (angulartics and google analytics) no longer used
  • adds an environment variable to the build process for open-ssl-ca
  • fixes the test so that we can test in branches and PRs without deploying to gh pages

This is a preliminary PR to maybe a bigger upgrade of node.js that is discussed in this issue: #445

@oscgonfer oscgonfer requested a review from pral2a July 31, 2023 19:10
@oscgonfer oscgonfer changed the title 445/node bower upgrade 445/bower-fix-and-cleanup Aug 13, 2023
@oscgonfer oscgonfer closed this Aug 13, 2023
@oscgonfer oscgonfer deleted the 445/node-bower-upgrade branch August 13, 2023 23:14
@oscgonfer oscgonfer restored the 445/node-bower-upgrade branch August 13, 2023 23:15
@oscgonfer oscgonfer reopened this Aug 13, 2023
@oscgonfer oscgonfer marked this pull request as draft August 13, 2023 23:31
@oscgonfer oscgonfer marked this pull request as ready for review August 20, 2023 22:09
@oscgonfer oscgonfer merged commit bce8e5f into master Oct 2, 2023
@oscgonfer oscgonfer deleted the 445/node-bower-upgrade branch March 18, 2024 18:58
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

Successfully merging this pull request may close these issues.

1 participant