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

Misc cleanup: Delete unused scripts and build logic #6492

Merged
merged 5 commits into from
Jan 20, 2022
Merged

Conversation

kmahone
Copy link
Member

@kmahone kmahone commented Dec 17, 2021

Cleaning up some of the cruft from the repo.

  • Deleted unused scripts from 'tools' directory. Most of these were related to the OS repo porting that we used to do, or were related to the Package ES build pool we used to use.

  • Removed all instances of BuildingWithBuildExe msbuild property. This was used when building in razzle in the OS repo.

  • Removed all instances of XES_DFSDROP. XES_OUTDIR which were used when we were building in Package ES system.

  • Other miscellaneous cleanup.

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Dec 17, 2021
@kmahone
Copy link
Member Author

kmahone commented Dec 17, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kmahone
Copy link
Member Author

kmahone commented Jan 6, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@StephenLPeters
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kmahone kmahone merged commit 62d9a6e into main Jan 20, 2022
@kmahone kmahone deleted the user/kmahone/cleanup branch January 20, 2022 23:16
@StephenLPeters StephenLPeters added area-DevInternal Internal build infrastructure, code cleanup, engineering efficiency team-Controls Issue for the Controls team and removed needs-triage Issue needs to be triaged by the area owners labels Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-DevInternal Internal build infrastructure, code cleanup, engineering efficiency team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants