Skip to content

Yarn Package Manager#1109

Merged
protonate merged 5 commits intomasterfrom
improvement/yarn
Apr 13, 2017
Merged

Yarn Package Manager#1109
protonate merged 5 commits intomasterfrom
improvement/yarn

Conversation

@protonate
Copy link
Collaborator

@protonate protonate commented Apr 6, 2017

Type of change

  • Build related changes

Description of change

Adds the Yarn package manager. NOTE: Yarn must be installed independently of this change and as fits the developer's environment. See install guide.

Usage is similar to NPM.

yarn.lock Outdated

ansi-escape-sequences@^3.0.0:
version "3.0.0"
resolved "https://npm.corp.appnexus.com/ansi-escape-sequences/-/ansi-escape-sequences-3.0.0.tgz#1c18394b6af9b76ff9a63509fa497669fd2ce53e"
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks to be a lot of internal npm references here. Need to fix.

Copy link
Collaborator Author

@protonate protonate Apr 7, 2017

Choose a reason for hiding this comment

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

oh I see you mean anx internal npm registry, yep got it.

Do you mean npm registry references? That is expected, Yarn is an npm registry client, not a replacement. registry.yarnpkg.com is a proxy for registry.npmjs.org.

@protonate
Copy link
Collaborator Author

@mkendall07 better now

@mkendall07
Copy link
Contributor

LGTM.

@mkendall07 mkendall07 requested a review from snapwich April 10, 2017 20:19
@mkendall07
Copy link
Contributor

@snapwich
please review

**Build standalone prebid.js**

- Clone the repo, run `npm install`
- Clone the repo, run `yarn install`
Copy link
Collaborator

Choose a reason for hiding this comment

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

Might want to add a link to the yarn project for those that are unfamiliar with it.

@protonate
Copy link
Collaborator Author

Note addressed, thanks for reviews.

@protonate protonate merged commit b913f96 into master Apr 13, 2017
outoftime added a commit to Genius/Prebid.js that referenced this pull request Apr 19, 2017
…built

* 'master' of https://github.com/prebid/Prebid.js: (38 commits)
  Add optional domain parameter to AdButler adapter (prebid#1078)
  Send transactionID to Criteo Services (prebid#1113)
  Fix `buildMasterVideoTagFromAdserverTag()` not selecting winning bid (prebid#1106)
  Remove placement size selection and filtering (prebid#1107)
  revert `srcdoc` change (prebid#1130)
  Add new Adapter- Beachfront Media (prebid#1062)
  Fixes SpringServe adapter (prebid#1101)
  Update Widespace request param (prebid#1098)
  - New Adapter: Innity (prebid#1074)
  Update Roxot prebid analytic adapter (prebid#1034)
  Yarn Package Manager (prebid#1109)
  allow writing into current document if prebid is loaded inside an iframe (prebid#1066)
  Adapter bug fix (prebid#1096)
  fix typo
  added pr review process and governance model (prebid#1103)
  added support for sampling in ga and base adapter, fixed up some tests (prebid#1011)
  Add Inneractive adapter (prebid#1048)
  Add alias freewheel-ssp to stickyadstv bidder adapter  (prebid#1043)
  Add Facebook Audience Network adapter (prebid#1068)
  Add Atomx support (prebid#1056)
  ...
@protonate protonate deleted the improvement/yarn branch May 5, 2017 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments