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

Fix broken Appveyor CI #1083

Merged
merged 1 commit into from
Jul 6, 2021
Merged

Conversation

KineticTheory
Copy link
Collaborator

Background

  • The repository that hosted the tools provided by win32-vs16-tools.zip was deleted.
  • I have added this file to the Draco/wiki repository so that we maintain full control over it.

Description of changes

  • The appveyor.yml file was updated to download the zip file from the new location.

Status

+ The repository that hosted the tools provided by `win32-vs16-tools.zip` was deleted.
+ I have added this file to the Draco/wiki repository so that we maintain full control over it.
+ The `appveyor.yml` file was updated to download the zip file from the new location.
@KineticTheory KineticTheory added this to the Draco-7_12_0 milestone Jul 6, 2021
@KineticTheory KineticTheory self-assigned this Jul 6, 2021
Copy link
Contributor

@RyanWollaeger RyanWollaeger left a comment

Choose a reason for hiding this comment

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

LGTM

@RyanWollaeger RyanWollaeger merged commit b456c8b into lanl:develop Jul 6, 2021
@KineticTheory
Copy link
Collaborator Author

LGTM

Unfortunately, there is still a problem. I'll look into to it.

@RyanWollaeger
Copy link
Contributor

LGTM

Unfortunately, there is still a problem. I'll look into to it.

Ah - sorry if I merged this prematurely. I noticed the Appveyor CI failed, but thought this might have to be merged first.

@KineticTheory KineticTheory deleted the fix_appveyor_ci branch July 9, 2021 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants