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 build github action #233

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

add build github action #233

wants to merge 5 commits into from

Conversation

jaydns
Copy link

@jaydns jaydns commented Oct 27, 2020

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

Copy link
Contributor

@MrDevBot MrDevBot left a comment

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

how to compile ? i have too much errors

@LightYagami28
Copy link

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

@alphaO4
Copy link

alphaO4 commented Sep 25, 2024

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

Is the repo maintained?

@alphaO4
Copy link

alphaO4 commented Sep 25, 2024

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

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