Skip to content

Update CI build and Readme#6

Open
illusionyy wants to merge 2 commits into
orbisdev:masterfrom
illusionyy:patch-1
Open

Update CI build and Readme#6
illusionyy wants to merge 2 commits into
orbisdev:masterfrom
illusionyy:patch-1

Conversation

@illusionyy

@illusionyy illusionyy commented Aug 21, 2022

Copy link
Copy Markdown
  • Add instruction on how to setup without building
  • Add instruction on how to install dotnet (ubuntu)
  • Update CI to make release using gh

Sample build https://github.com/illusion0001/orbisdev/releases/tag/1.037
Readme: https://github.com/illusion0001/orbisdev/tree/patch-1#readme

@illusionyy illusionyy force-pushed the patch-1 branch 2 times, most recently from 746c69d to 250aab8 Compare August 22, 2022 20:27
- Add instruction on how to setup without building
- Add instruction on how to install dotnet (ubuntu)
- Update CI to make release using `gh`
Comment thread .github/workflows/compilation.yml Outdated
export ORBISDEV=$PWD/orbisdev
export PATH=$ORBISDEV/bin:$PATH
./build-all.sh
- name: Build stage 1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why are you running here in separate stages?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The time to run the individual stages can be monitored and if anything fails, it's easier to fix them instead of going through thousands line of log if running them all at once.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nah.... when it fails it will show at the very end of the log what stage failed anyway.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Ok I'll revert it

Comment thread .github/workflows/compilation.yml Outdated
paths-ignore:
- '*.md'
pull_request:
paths-ignore:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Sometimes we do dummy changes in the Readme to trigger a new CI execution, either you add a way of manually launching them, or remove that lines

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Added workflow_dispatch if you want to trigger a workflow manually.

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