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

update various docs to reflect new location of repo (#12)_8(PENDING Merge later) #272

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sahithiacn
Copy link
Contributor

@sahithiacn sahithiacn commented Dec 6, 2022

Check all code comments.

@sahithiacn
Copy link
Contributor Author

/canary

bors-diem pushed a commit that referenced this pull request Dec 6, 2022
@bors-diem
Copy link

☀️ Canary successful

@KoushikGavini
Copy link

Hi,

Can we add more description to this task. Not sufficient.

@KoushikGavini
Copy link

@sahithiacn please add more detailed test details and coverage

To contribute to the Diem Core implementation, first start with the proper
development copy.

To get the development installation with all the necessary dependencies for
linting, testing, and building the documentation, run the following:
```bash
git clone https://github.com/diem/diem.git
cd diem
git clone https://github.com/move-language/move.git

Choose a reason for hiding this comment

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

---

## ❓ Questions and Help

### Please note that this issue tracker is not a help form and this issue will be closed.

Please contact the development team on [Discourse](https://community.libra.org)
Please contact the development team on [Discord](https://discord.com/invite/M95qX3KnG8)

Choose a reason for hiding this comment

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

check discord link. it should be changed to Diems development team

@@ -1,10 +1,10 @@
---
name: ❓ Questions/Help
about: If you have questions, please check Discourse
about: If you have questions, please check Discord

Choose a reason for hiding this comment

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

check below comments

@@ -10,7 +10,7 @@ assignees: ''
# 🐛 Bug

<!-- A clear and concise description of what the bug is.
To report a security issue, please email security@diem.com. -->
To report a security issue, please email build@mystenlabs.com. -->

Choose a reason for hiding this comment

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

check this change "email id"

@sahithiacn
Copy link
Contributor Author

/land

@bors-diem
Copy link

@sahithiacn ❗ This PR is still missing approvals, unable to queue for landing

@sahithiacn sahithiacn changed the title update various docs to reflect new location of repo (#12)_8 update various docs to reflect new location of repo (#12)_8(PENDING Merge later) Dec 9, 2022
@sahithiacn sahithiacn changed the title update various docs to reflect new location of repo (#12)_8(PENDING Merge later) update various docs to reflect new location of repo (#12)_7(PENDING Merge later) Dec 22, 2022
@sahithiacn sahithiacn changed the title update various docs to reflect new location of repo (#12)_7(PENDING Merge later) update various docs to reflect new location of repo (#12)_8(PENDING Merge later) Dec 22, 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.

5 participants