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

lookup: re-add bluebird #989

Closed
wants to merge 2 commits into from

Conversation

alfonsograziano
Copy link
Contributor

@alfonsograziano alfonsograziano commented Sep 29, 2023

Checklist
  • npm test passes
  • contribution guidelines followed
    here

@Trott
Copy link
Member

Trott commented Sep 29, 2023

It will take a while for the job to start, but https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/3285/ will hopefully have bluebird results.

Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

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

Bluebird passes locally but fails in Jenkins CI.

lib/lookup.json Outdated Show resolved Hide resolved
@Trott
Copy link
Member

Trott commented Sep 29, 2023

Running again with ANSI stipping enabled, just in case that's somehow the source of the problem. https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/3286/

@alfonsograziano
Copy link
Contributor Author

alfonsograziano commented Sep 29, 2023

Still failing, to be honest I'm not sure if it is a legittimate issue or a false positive. Is there any process in place for now to deal with these cases? (I mean when locally it works but fails on the CI pipeline)

@Trott
Copy link
Member

Trott commented Sep 29, 2023

The ANSI-stripping didn't do anything either. Harumph.

(I mean when locally it works but fails on the CI pipeline)

I wish I had an answer. I have no idea why this passes locally and fails everywhere in CI. Must be something in the CI configuration somewhere, but I'm not even sure what to look for. Suggestions welcome.....

@nodejs/build

TBH, unless bluebird is important to you personally, I'm not sure this is worth a lot of effort. Bluebird hasn't seen a new version in 4 years and is likely mostly obsolete at this point. Its current download stats are an artifact of its enormous popularity and utility in the past.

@alfonsograziano
Copy link
Contributor Author

Good point, probably is not worth to add this package. Is definitely weird that the pipeline is failing on the CI. If the same happens with other packages, then probably is worth to investigate.

I'll close this PR for now and I'll focus on other packages that we should add due to their popularity in the ecosystem.

Thanks for your support, really appreciated 😃

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.

2 participants