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 issue experienced in #649 #843

Merged
merged 3 commits into from
Mar 20, 2020
Merged

Conversation

wilsonbn
Copy link
Contributor

@wilsonbn wilsonbn commented Feb 19, 2020

What kind of change does this PR introduce?

Moves the RefitInternalGenerated using declaration to the top of the GeneratedInterfaceStubTemplate.mustache file with others

What is the current behavior?

More detail can be found in #649. The generated template cannot find RefitInternalGenerated or Preserve and other classes, properties and attributes related to the namespace.

What is the new behavior?

I found moving the using allowed my 3.1 project to build without the errors.

What might this PR break?
I don't believe this change will cause anything to break. I have not seen anything else within the project with the name RefitInternalGenerated.

Please check if the PR fulfills these requirements

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Other information:
First PR for me to this project. If I am wrong about the requirements for this change being complete through existing tests and documentation of this file please direct me and I will correct.

I found moving the RefitInternalGenerated using declaration to the top allowed my 3.1 project to build without the errors found in bug reactiveui#649
RefitInternalGenerated using declaration moved
@clairernovotny clairernovotny merged commit 9b25e1f into reactiveui:master Mar 20, 2020
@lock lock bot locked and limited conversation to collaborators Jun 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants