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

chore: lfs file rewrite #258

Conversation

dkilgore-eightfold
Copy link
Contributor

@dkilgore-eightfold dkilgore-eightfold commented Jul 25, 2022

SUMMARY:

The size of the *.snap files located in the commit history of the Octuple repo are quite large, resulting in the repo reaching its storage quota before the end of the month whenever git lfs is not available. This change uses git filter-repo to remove those large files by type from the repo history using the following command:

git filter-repo --force --path-glob '*.snap' --invert-paths

Without this change, upon cloning, committing, pushing or pulling developers will encounter the following git lfs error when there is no available data plan:

batch response: This repository is over its data quota. Account responsible for LFS bandwidth should purchase more data packs to restore access.

Once this change is merged into main, then main-next and v1.x-hotfix should also take this change. Upon completing this task we may then follow steps to git lfs uninstall for the main repo ahead of merging main-next into main .

References:

newren/git-filter-repo

How to Remove a Large File from Commit History in Git?

Git - Remove All of a Certain Type of File from the Repository

dependabot bot and others added 30 commits March 23, 2022 13:32
Bumps [minimist](https://github.com/substack/minimist) from 1.2.5 to 1.2.6.
- [Release notes](https://github.com/substack/minimist/releases)
- [Commits](https://github.com/substack/minimist/compare/1.2.5...1.2.6)

---
updated-dependencies:
- dependency-name: minimist
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Updates button component to function component, adds base component.

* Updates button components, adds BaseButton and allowDisabledFocus prop, updates styles, uses upcoming Icon component.

* Adds props commentary

* Updates css module references

* Includes button scss module

* Removes component css module

* Adds foreground color detection helper method invertForegroundColor to ensure the foreground color is legible when an arbitrary cobranding color is provided as the button background color.

* Adds Flex Button Size as default, makes button responsive.

* Updates css modules properties to convert snake-case to camelCase, Adds support for updating css variables from components, adds 2 icons, adds internal props extension for button.

* Breakpoint logic into its own hook (useMatchMedia()), cleans things up a bit.

* Address PR feedback, adds id prop to button and icon.

* Adds Button functionality UTs
… 16.14.2 (EightfoldAI#15)

* Fixes missing d.ts SASS module files for prod build using latest Node 16.14.2

* Updates gitignore, removes definitely typed files
The dark selector was missing from the button scss module
This change adds it with a stubbed selector and property
…dAI#19)

* fix: button: refines button prop and adds form type support

Renames private ButtonType prop to ButtonMode to enable type for button

* fix: address pr feedback: removes from actions, renames type
…ssage linter and updates readmes (EightfoldAI#18)

* chore(release): 0.0.1

* chore(release): 0.0.1

* chore(release): 0.0.1

* feat: changelog readmes commit linting: adds changelog scripts, message linter, updates readmes

Ensures commit messages are meaningful, adds changeelog and REAMDEs

* fix: typo: fixes readme typo and aid clarification

The subject line should not be sentence cased
Adds more details about commit messages

* fix: edit readmes: adjusts the information funnel from the main readme

Better surface contribution guidelines from main readme

* fix: removes duplicates: removes duped readmes

* fix: asset swap: fixes referenced asset context
* fix: readme: fixes asset and documentation urls

the readme was using relative urls, this change fixes it

* fix: husky: removes postinstall in favor of prepare to work with npm
…tfoldAI#29)

* fix: react: ignore react in production build to avoid conflicts

* chore: version: bump version in package json

* chore: npm: updates npm ignore
Fixes break by removing unused selectors
dkilgore-eightfold and others added 25 commits July 8, 2022 10:12
* feat: picker: adds date and time picker components

initial commit that adds internal components

* chore: datepicker: adds date picker hoc, loc, and uts

moves moment to peer deps

* chore: datepicker: add missing css selector

* chore: datepicker: remove moment and address pr feedback

* chore: datepicker: adjust clear icon y position

* chore: datepicker: fix ci build linter errors

* chore: datepicker: fix ci linter errors part two

* chore: datepicker: fix ci linter errors part three

* chore: datepicker: fix ci linter part four

* chore: datepicker: fix ci linter part five

* chore: datepicker: fix ci linter errors part six

* chore: datepicker: fix ci linter errors part seven

* chore: datepicker: fix ci linter errors part eight

* chore: linter: revert slider change

* chore: datepicker: revert tab change
## SUMMARY:
adds mdi "auto-fix" icon
(required by careerhub profile assistant)

## CHANGE TYPE:

-   [ ] Bugfix Pull Request
-   [x] Feature Pull Request

## TEST PLAN:
in storybook should observe MDIAutoFix icon
<img width="1206" alt="image" src="https://user-images.githubusercontent.com/60897218/179025458-aa939a00-afbd-4e70-af5a-29b65db2e573.png">
* feat: input: add underline variant

* chore: input: add disabled styles and refine shapes
* feat: input: add underline variant

* chore: input: add disabled styles and refine shapes

* feat: input: add inline prop

fix underline with icon and button and typos

* chore: input: remove input type specificity
* fix: select: Move chevron indicator to the right side

also adds select shapes and sizes mapped to input

* chore: input: alphabetize search box props
…EightfoldAI#243)

* chore: test: update snapshots

* chore: test: update snapshots

* fix: select: fixes pills, filtering, empty text, and spinner position

* chore: test: update snapshots

* chore: select: update empty text layout to match spinner height

* chore: select: pixel push empty text container
Bumps [terser](https://github.com/terser/terser) from 4.8.0 to 4.8.1.
- [Release notes](https://github.com/terser/terser/releases)
- [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md)
- [Commits](https://github.com/terser/terser/commits)

---
updated-dependencies:
- dependency-name: terser
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
… as intended (EightfoldAI#253)

* fix: radiobutton: multiple radio buttons on the same page doesnt work as intended

also adds header action buttons to panel and normalizes close button props

* chore: radiobutton: update snapshots
@dkilgore-eightfold dkilgore-eightfold changed the title Dkilgore eightfold/lfs file rewrite chore: lfs file rewrite Jul 25, 2022
Copy link
Contributor

@jhoward-eightfold jhoward-eightfold left a comment

Choose a reason for hiding this comment

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

Just going on record that I am NOT reviewing 600,000+ lines of code. Trusting that what you are doing here is appropriate 😆

@dkilgore-eightfold dkilgore-eightfold merged commit 9965531 into EightfoldAI:main Jul 25, 2022
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.

10 participants