Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Removed import-module $helperModule as it is not needed.#3814

Merged
daxian-dbw merged 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
adityapatwardhan:FixDailyBuildadityapatwardhan/PowerShell:FixDailyBuildCopy head branch name to clipboard
May 19, 2017
Merged

Removed import-module $helperModule as it is not needed.#3814
daxian-dbw merged 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
adityapatwardhan:FixDailyBuildadityapatwardhan/PowerShell:FixDailyBuildCopy head branch name to clipboard

Conversation

@adityapatwardhan

Copy link
Copy Markdown
Member

$helperModule is not defined anymore after refactoring changes. Import-Module is not necessary, hence removed.
This is was not caught by CI as these are Feature tests.
This caused the daily build to be broken.

Related to changes done in: #3456

$helperModule is not defined anymore after refactoring changes. Import-Module is not necessary, hence removed.
This is was not caught by CI as these are Feature tests.
@adityapatwardhan

Copy link
Copy Markdown
Member Author

@daxian-dbw Can you have a look? This is breaking daily build.

@SteveL-MSFT SteveL-MSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@iSazonov iSazonov left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM.

@adityapatwardhan

Copy link
Copy Markdown
Member Author

@PowerShell/powershell-maintainers This is ready to merge. This fixes the daily build.

@daxian-dbw daxian-dbw merged commit f201782 into PowerShell:master May 19, 2017
@adityapatwardhan adityapatwardhan deleted the FixDailyBuild branch May 19, 2017 18:03
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.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.