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

Add .github/workflow/mirror_to_gitee.yml #475

Merged
merged 20 commits into from
Jul 28, 2021
Merged

Conversation

tuoping
Copy link
Collaborator

@tuoping tuoping commented Jul 21, 2021

I found the problem. I've been invoking action in my personal repository: github.com/tuoping/dpgen. So the ssh-private-key in organization deepmodeling cannot be visited. We have to test the action after PR.

@tuoping tuoping requested a review from njzjz July 21, 2021 04:12
@codecov-commenter
Copy link

codecov-commenter commented Jul 21, 2021

Codecov Report

Merging #475 (f0ccfce) into devel (86afc5b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            devel     #475   +/-   ##
=======================================
  Coverage   32.97%   32.97%           
=======================================
  Files          86       86           
  Lines       14736    14736           
=======================================
  Hits         4859     4859           
  Misses       9877     9877           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 86afc5b...f0ccfce. Read the comment docs.

@tuoping
Copy link
Collaborator Author

tuoping commented Jul 21, 2021

In addition, may I ask how did you enable to sync issue from gitee to github?

.github/workflows/mirror_gitee.yml Outdated Show resolved Hide resolved
@njzjz
Copy link
Member

njzjz commented Jul 21, 2021

In addition, may I ask how did you enable to sync issue from gitee to github?

A simple method... Set up this bot and then forward the Gitee notification emails to it. (Create a rule in Gmail)

Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@tuoping
Copy link
Collaborator Author

tuoping commented Jul 21, 2021

Sorry about that. But this action still failed with the same error message in my repository tuoping/dpgen. I still think the action need to be invoked in deepmodeling/dpgen.

@njzjz
Copy link
Member

njzjz commented Jul 21, 2021

Did you set the secret in your own repo?

@tuoping
Copy link
Collaborator Author

tuoping commented Jul 23, 2021

The point of this is to test the secret in organization. That's not a bad idea though, to set the secret in my own repo and PR with the mirror_to_gitee.yml file. This way the secret in organization will be of no use.

@njzjz
Copy link
Member

njzjz commented Jul 23, 2021

If you want to test the secret in organization, you need to push the branch into the organization repository. (or ask someone who has the access to help you)

@amcadmus amcadmus merged commit 7f86e32 into deepmodeling:devel Jul 28, 2021
Cloudac7 pushed a commit to Cloudac7/dpgen that referenced this pull request Dec 1, 2021
* make doc directory for toy models; deploy doc in a branch gh-doc

* add contribution guide to doc; add conf.py in doc

* move doc dependence from requirements to doc/requirements

* deploy on readthedocs instead of github

* add mirror_gitee.yml in workflow

* test action at push

* test action at push

* test action at push

* test action at push

* test action mirror

* test action mirror

* typo

* use https in mirror workflow

* use https in mirror workflow

* use https in mirror workflow

* mirror to gitee

* mirror to gitee

* Update .github/workflows/mirror_gitee.yml

Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>

Co-authored-by: tuoping <abby@DESKTOP-LV5KL0D.localdomain>
Co-authored-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
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.

4 participants