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

[APM] Update tsconfig.json template in optimization script #58731

Merged
merged 2 commits into from
Feb 28, 2020

Conversation

dgieselaar
Copy link
Member

The location of the cypress tests has changed, and those files are no longer excluded in the APM type check. This change updates the tsconfig.json template to exclude the new location.

The location of the cypress tests has changed, and those files are no longer excluded in the APM type check. This change updates the tsconfig.json template to exclude the new location.
@dgieselaar dgieselaar added release_note:skip Skip the PR/issue when compiling release notes v7.7.0 labels Feb 27, 2020
@dgieselaar dgieselaar requested a review from a team as a code owner February 27, 2020 15:09
@sorenlouv
Copy link
Member

Thanks for fixing this.
While you are add it, can you also fix the link in the readme

"Cypress integration tests" should link to https://github.com/elastic/kibana/blob/d27b6c05d33c3274ba43986ba3a722e26b69d2b2/x-pack/legacy/plugins/apm/e2e/README.md

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@dgieselaar dgieselaar merged commit f86c758 into elastic:master Feb 28, 2020
@dgieselaar dgieselaar deleted the update-tsconfig-template branch February 28, 2020 15:32
dgieselaar added a commit to dgieselaar/kibana that referenced this pull request Feb 28, 2020
…8731)

* [APM] Update tsconfig.json template in optimization script

The location of the cypress tests has changed, and those files are no longer excluded in the APM type check. This change updates the tsconfig.json template to exclude the new location.

* Update link in readme
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Feb 29, 2020
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Feb 29, 2020
dgieselaar added a commit that referenced this pull request Feb 29, 2020
…58885)

* [APM] Update tsconfig.json template in optimization script

The location of the cypress tests has changed, and those files are no longer excluded in the APM type check. This change updates the tsconfig.json template to exclude the new location.

* Update link in readme

Co-authored-by: Elastic Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v7.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants