Skip to content

Conversation

@RogPodge
Copy link
Contributor

Overview

When deploying to Oculus the 618 error will still pop up, this adds that code to the csc to fix that issue.

Fixed some whitespace that was introduced accidentally.

@RogPodge
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

List<string> warningNumbers = new List<string>();

// List of new warning numbers to add to the csc file
// Empty as of Oculus Integration 39.0 https://developer.oculus.com/downloads/package/unity-integration/39.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this comment still accurate? I also still have an open comment (along with some others) about this on a previous PR: #10576 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Addressed the comments in the other PR.

@RogPodge
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@RogPodge RogPodge requested a review from keveleigh May 16, 2022 20:43
@RogPodge RogPodge merged commit 7bfc823 into microsoft:prerelease/2.8.0 May 16, 2022
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