Skip to content

add build github action#233

Open
jaydns wants to merge 5 commits into
NYAN-x-CAT:masterfrom
jaydns:master
Open

add build github action#233
jaydns wants to merge 5 commits into
NYAN-x-CAT:masterfrom
jaydns:master

Conversation

@jaydns

@jaydns jaydns commented Oct 27, 2020

Copy link
Copy Markdown

Add a GitHub Action to automatically build AsyncRAT.
This has the added benefit of building every PR, allowing PR builds to be tested quickly.

@MrDevBot MrDevBot left a comment

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.

You really shouldn't be using a hard coded path for a project build script
D:\a\AsyncRAT-C-Sharp\AsyncRAT-C-Sharp\AsyncRAT-C#\Binaries${{ matrix.configuration }}\

@matoqueimado

Copy link
Copy Markdown

how to compile ? i have too much errors

@LightYagami28

Copy link
Copy Markdown

Hello developer i made a GitHub Fork with all your upgrade. And then i made in net 9.0

@alphaO4

alphaO4 commented Sep 25, 2024

Copy link
Copy Markdown

You really shouldn't be using a hard coded path for a project build script

D:\a\AsyncRAT-C-Sharp\AsyncRAT-C-Sharp\AsyncRAT-C#\Binaries${{ matrix.configuration }}\

Also wouldn't this only be triggered when a push happens to master? And not like OP says, in every PR

@LightYagami28

Copy link
Copy Markdown

Is the repo maintained?

@alphaO4

alphaO4 commented Sep 25, 2024

Copy link
Copy Markdown

Is the repo maintained?

Seeing the last update was a minors one and 11 months ago, not really. It still works fine for most use cases (I.e. Remote administration of your own machines), but I suspect when it breaks, it won't receive a new update. Sad to see, but understandable.

@LightYagami28

Copy link
Copy Markdown

I'm trying hard to improve the code but with great difficulty

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