Skip to content

fix(pd-store): correct license in hugegraph-store#2447

Merged
imbajin merged 3 commits intopd-storefrom
store-license-header
Feb 20, 2024
Merged

fix(pd-store): correct license in hugegraph-store#2447
imbajin merged 3 commits intopd-storefrom
store-license-header

Conversation

@VGalaxies
Copy link
Copy Markdown
Contributor

@VGalaxies VGalaxies commented Feb 6, 2024 •
edited by imbajin
Loading

subtask of #2265

follow up #2435 on pd-store branch

NOTE: this PR removed some meaningless configuration files

hugegraph-store/build-change.sh
hugegraph-store/build-pre.sh
hugegraph-store/build.sh
hugegraph-store/ci.yml
hugegraph-store/deploy-release.sh
hugegraph-store/deploy-snapshot.sh
hugegraph-store/hg-store-cli/deploy-snapshot.sh
hugegraph-store/hg-store-client/deploy-snapshot.sh
hugegraph-store/hg-store-common/mvnw
hugegraph-store/hg-store-common/mvnw.cmd
hugegraph-store/hg-store-core/build.sh
hugegraph-store/hg-store-grpc/build.sh
hugegraph-store/hg-store-node/src/assembly/static/bin/app.sh
hugegraph-store/hg-store-node/src/assembly/static/bin/hg-get-release.sh
hugegraph-store/hg-store-node/src/assembly/static/bin/hg-update.sh
hugegraph-store/init.sh
hugegraph-store/local-release.sh
hugegraph-store/run.sh
hugegraph-store/settings.xml

marked them for future need

@VGalaxies VGalaxies added the store Store module label Feb 6, 2024
@VGalaxies VGalaxies requested a review from imbajin February 6, 2024 13:56
@VGalaxies VGalaxies self-assigned this Feb 6, 2024
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. ci-cd Build or deploy labels Feb 6, 2024
@VGalaxies VGalaxies mentioned this pull request Feb 6, 2024
Closed
26 tasks
@VGalaxies
Copy link
Copy Markdown
Contributor Author

VGalaxies commented Feb 6, 2024 •
edited
Loading

@Pengzna could also help check it by following the guide from Project Migration/Package Name Change + IDEA Configuration Instructions (Complete Version)~

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 6, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (pd-store@7b14857). Click here to learn what that means.

Additional details and impacted files 
@@             Coverage Diff             @@
##             pd-store    #2447   +/-   ##
===========================================
  Coverage            ?   55.27%           
  Complexity          ?     1901           
===========================================
  Files               ?      718           
  Lines               ?    58293           
  Branches            ?     7483           
===========================================
  Hits                ?    32219           
  Misses              ?    22903           
  Partials            ?     3171

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

imbajin
imbajin approved these changes Feb 7, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@imbajin imbajin left a comment •
edited
Loading

Choose a reason for hiding this comment

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

checked almost all files, better to mark deleted files(PR link) in a separate doc

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 7, 2024
@imbajin imbajin merged commit e4bd644 into pd-store Feb 20, 2024
@imbajin imbajin deleted the store-license-header branch February 20, 2024 10:25
@VGalaxies VGalaxies mentioned this pull request Mar 16, 2024
Closed
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@imbajin imbajin imbajin approved these changes

Assignees

@VGalaxies VGalaxies

Labels

ci-cd Build or deploy lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files. store Store module

Projects

HugeGraph PD-Store Tasks
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@VGalaxies @imbajin