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

[OSS101] Complete task 6: Refactor the columns of the project and developer leaderboard #121

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

MarigWeizhi
Copy link

Brief Information

Add the line chart of openrank's recent trend changes

This pull request is in the type of (more info about types):

  • build
  • ci
  • docs
  • feat
  • fix
  • perf
  • refactor
  • test

Related issues: #116

Details

Add the line chart of openrank's recent trend changes
Temporarily generate trend data randomly according to the changes, the data interface supports trendData and then enter it into the Trend component.

Checklist

@andyhuang18
Copy link
Collaborator

Refer to contribution. You should run npm run prettier to unify the code format after completing the task. This problem caused the workflow to fail. Please modify and commit the code in time.

@MarigWeizhi
Copy link
Author

Refer to contribution. You should run npm run prettier to unify the code format after completing the task. This problem caused the workflow to fail. Please modify and commit the code in time.

The last workflow reported an error with pnpm-lock.yaml being out of sync with package.json, I fixed it!

@andyhuang18
Copy link
Collaborator

Still have problem, please check the error message.

image

@MarigWeizhi
Copy link
Author

Still have problem, please check the error message.

image

updated pnpm-lock.yaml for the nodejs 16.x and pnpm 8.x

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