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(tools): typescript composite #2315

Merged
merged 4 commits into from Jan 16, 2023
Merged

fix(tools): typescript composite #2315

merged 4 commits into from Jan 16, 2023

Conversation

zeroedin
Copy link
Collaborator

What I did

pfe-tools: Set typescript compilerOptions composite: true

pfe-tooltip: Added return type for anonymous function for content in constructor

@zeroedin zeroedin added the 1:1 Aligning components with PatternFly v4 label Jan 16, 2023
@zeroedin zeroedin self-assigned this Jan 16, 2023
@changeset-bot
Copy link

changeset-bot bot commented Jan 16, 2023

🦋 Changeset detected

Latest commit: bd6f510

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@patternfly/pfe-tooltip Patch
@patternfly/pfe-tools Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@zeroedin zeroedin requested a review from heyMP January 16, 2023 20:16
@github-actions github-actions bot added functionality Functionality, typically pertaining to the JavaScript. tools Development and build tools labels Jan 16, 2023
@github-actions github-actions bot added this to In progress in Workflow Jan 16, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jan 16, 2023

Deploy Preview for patternfly-elements ready!

Name Link
🔨 Latest commit 8ac991e
😎 Deploy Preview https://deploy-preview-2315--patternfly-elements.netlify.app/

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot added the AT passed Automated testing has passed label Jan 16, 2023
Copy link
Contributor

@heyMP heyMP left a comment

Choose a reason for hiding this comment

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

@bennypowers I'm going to go ahead an approve this PR so it fixes the build errors. Steven and I were guessing that there might be a better TS config solution.

Workflow automation moved this from In progress to Approved Jan 16, 2023
@heyMP heyMP enabled auto-merge (squash) January 16, 2023 21:10
@heyMP heyMP merged commit 457eaa9 into main Jan 16, 2023
15 checks passed
Workflow automation moved this from Approved to Done Jan 16, 2023
@heyMP heyMP deleted the fix/tools/typescript-composite branch January 16, 2023 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1:1 Aligning components with PatternFly v4 AT passed Automated testing has passed functionality Functionality, typically pertaining to the JavaScript. ready to merge tools Development and build tools
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants