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

Renames to CaminoJS #6

Merged
merged 4 commits into from
Oct 26, 2022
Merged

Renames to CaminoJS #6

merged 4 commits into from
Oct 26, 2022

Conversation

Chriserus
Copy link

No description provided.

@Ayagoumi Ayagoumi self-requested a review October 26, 2022 09:58

For testing end to end, you need to have a local network running. This can easily be achieved by using Avalanche Network Runner. You can follow the steps [here](https://docs.avax.network/quickstart/network-runner) to setup a local network.
For testing end to end, you need to have a local network running. This can easily be achieved by using Camino Network Runner. You can follow the steps [here](https://docs.avax.network/quickstart/network-runner) to setup a local network.

Choose a reason for hiding this comment

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

docs.avax should be changes to the one in docs CAM

Confirm that everything worked by running `yarn start` and viewing the new CaminoJS docs on `localhost`. If that is successful next run `yarn build` to confirm that docusaurus is able to properly build. Once that is successful, branch off of `master`, commit your changes and push them on your new feature branch to the remote `avalanche-docs` repo and create a PR.

Choose a reason for hiding this comment

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

avalanche-docs should be changes to camino-docs

@@ -45,4 +45,4 @@ Now copy the `api.md` file into the `/path/to/docs/apis/tools/avalanchejs/` dire

Next copy 3 directories, `classes/`, `interfaces/` and `modules/` into the `/path/to/avalanchejs-docs/` directory of your local `avalanchejs-docs/` repo.

Choose a reason for hiding this comment

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

avalanche-docs should be changed to camino-docs

@@ -29,23 +29,23 @@ jobs:
mkdir -p $GOPATH/$AVAPATH

Choose a reason for hiding this comment

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

$AVAPATH should also change to $CAMPATH

Copy link

@Ayagoumi Ayagoumi left a comment

Choose a reason for hiding this comment

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

There is some minor changes to be done

Copy link

@Ayagoumi Ayagoumi left a comment

Choose a reason for hiding this comment

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

some more minor changes.


## Overview

We use [typedoc](https://typedoc.org) to turn comments in our typescript into markdown pages which can be copied into [the Avalanche Docs repo](https://github.com/ava-labs/avalanche-docs) and [the AvalancheJS Docs repo](https://github.com/ava-labs/avalanchejs-docs). Each time a new version of AvalancheJS gets deployed to [npm](https://www.npmjs.com/package/avalanche) then typedocs should be regenerated and pasted into the both docs repo previously mentioned.
We use [typedoc](https://typedoc.org) to turn comments in our typescript into markdown pages which can be copied into [the CaminoJS Docs repo](https://github.com/chain4travel/caminojs-docs). Each time a new version of CaminoJS gets deployed to [npm](https://www.npmjs.com/package/@c4tplatform/caminojs) then typedocs should be regenerated and pasted into the both docs repo previously mentioned.

Choose a reason for hiding this comment

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

Camino Docs doesn't exist either change this or we should change the name of the repo CaminoJs Docs

* Nest the list items per their parent category
* Clean up the body copy. Ex: `API-AVM` -> `AVM` and `API-AVM-Credentials` -> `Credentials`

## Copy to Docs Repo

Now copy the `api.md` file into the `/path/to/docs/apis/tools/avalanchejs/` directory of your local `avalanche-docs/` repo.
Now copy the `api.md` file into the `/path/to/docs/apis/tools/avalanchejs/` directory of your local `camino-docs/` repo.

Choose a reason for hiding this comment

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

also these paths need to updating

* Nest the list items per their parent category
* Clean up the body copy. Ex: `API-AVM` -> `AVM` and `API-AVM-Credentials` -> `Credentials`

## Copy to Docs Repo

Now copy the `api.md` file into the `/path/to/docs/apis/tools/avalanchejs/` directory of your local `avalanche-docs/` repo.
Now copy the `api.md` file into the `/path/to/docs/apis/tools/avalanchejs/` directory of your local `camino-docs/` repo.

Next copy 3 directories, `classes/`, `interfaces/` and `modules/` into the `/path/to/avalanchejs-docs/` directory of your local `avalanchejs-docs/` repo.

Choose a reason for hiding this comment

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

avalanchejs-docs/ should be changed to caminojs-docs/

Confirm that everything worked by running `yarn start` and viewing the new CaminoJS docs on `localhost`. If that is successful next run `yarn build` to confirm that docusaurus is able to properly build. Once that is successful, branch off of `master`, commit your changes and push them on your new feature branch to the remote `camino-docs` repo and create a PR.

Choose a reason for hiding this comment

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

if the docs repo was renamed this should be changed too.

cd camino-network-runner
git checkout v0.1.4
go run ./examples/local/fivenodenetwork/main.go &
- name: AvalancheJS E2E Test
- name: CaminoJS E2E Test
env:
AVALANCHEGO_IP: localhost

Choose a reason for hiding this comment

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

AVALANCHEGO_IP and AVALANCHEGO_PORT should be changed to CAMINOGO_IP and CAMINO_PORT

Copy link

@Ayagoumi Ayagoumi left a comment

Choose a reason for hiding this comment

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

Looks Good.

@Chriserus Chriserus merged commit 1aaf11d into chain4travel Oct 26, 2022
@c4t-ag c4t-ag deleted the caminojs-renames branch November 22, 2022 12:14
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