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

Test and fix build check #64

Merged
merged 6 commits into from
Feb 21, 2024
Merged

Test and fix build check #64

merged 6 commits into from
Feb 21, 2024

Conversation

sebastienros
Copy link
Contributor

No description provided.

@sebastienros sebastienros changed the title Update AstBuilder.cs Test and fix build check Feb 1, 2024
@sebastienros
Copy link
Contributor Author

@rivantsov it's working, and showing that one test is failing. I will use this PR to fix the test.

The appveyor integration can only be removed from appveyor itself.

@rivantsov
Copy link
Contributor

appveyor - I did not add it. I followed links, tried to login there thru github account, but it gave me 'blank' account, no existing integrations. don't remember who added this. Is there a way now to disable it? and by the way, I don't mind disabling the build itself

@sebastienros
Copy link
Contributor Author

@yallie you seem to own the appveyor project that is linked to this repository? Can you please delete it now that we have integrated GitHub actions?

@yallie
Copy link
Member

yallie commented Feb 3, 2024

Hi!
I deleted the linked Appveyor project, should be no Appveyor checks on PRs anymore.
Thanks @sebastienros for pointing it out!

@sebastienros sebastienros merged commit 22be69e into master Feb 21, 2024
2 checks passed
@sebastienros sebastienros deleted the sebros/testcheck branch February 21, 2024 01:32
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.

3 participants