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.

Sign up for GitHub

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

Jump to bottom

Feat: Rearrange repo as a monorepo #30

Merged
merged 22 commits into from
Feb 9, 2024
Merged

Feat: Rearrange repo as a monorepo #30

merged 22 commits into from
Feb 9, 2024

Conversation

carlosmondra
Copy link
Member

@carlosmondra carlosmondra commented Feb 8, 2024 •
edited
Loading

This PR:

  • Separates different packages to publish them as different libraries in npm
  • Converts both packages to use import statements

@carlosmondra
Feat: Rearrange repo as a monorepo
8e4d89d
@carlosmondra
Fix: Include missing eslint dev dependency
fdc5de1
@carlosmondra
Refactor: Rename abstract extractor
812be59
@carlosmondra
Merge branch 'master' into feature/monorepo
9972bf2
@carlosmondra
Fix: Remove this keyword
5016874
@carlosmondra
Chore: Update puppeteer
d7bd12e
@carlosmondra
Test: Remove commented code
497be98
@carlosmondra
Chore: Increase client version
a39396a
@carlosmondra
Feat: Increase client package version
e83fe20
@carlosmondra
Chore: Add READMEs
e90a5bd
@carlosmondra
Publish
8a48834 
 - @openreview/client@0.0.26
 - @openreview/meta-extraction@0.0.1
@carlosmondra
Feat: Update command to publish to the npm registry
5fbcf71
@carlosmondra
Chore: Fix some configurations
4a36d16
@carlosmondra
Chore: Perform clean install
ab3f942
@carlosmondra
Chore: Use npx
d71454d
@carlosmondra
Chore: Perform install
79247c1
@carlosmondra
Chore: Remove linting
8884a04
@carlosmondra
Fix: Install optional dependencies
98420f6
@carlosmondra
Chore: Add linting
507765d
@carlosmondra
Chore: Remove lerna from dependecies
1d3505a
@carlosmondra
Revert "Chore: Remove lerna from dependecies"
279b7b2 
This reverts commit 1d3505a.
@carlosmondra carlosmondra marked this pull request as ready for review February 9, 2024 15:54
@carlosmondra
Chore: Remove lerna versioning
4acd02d 
We will do the versioning manually
@xkopenreview xkopenreview merged commit 7314633 into master Feb 9, 2024
1 of 2 checks passed
@carlosmondra carlosmondra deleted the feature/monorepo branch February 9, 2024 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xkopenreview xkopenreview xkopenreview approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@carlosmondra @xkopenreview