Skip to content

[internal] Start of JS backend for target types and tailor goal#15442

Merged
Eric-Arellano merged 5 commits intopantsbuild:mainfrom
sureshjoshi:14190-js-ts-support
May 13, 2022
Merged

[internal] Start of JS backend for target types and tailor goal#15442
Eric-Arellano merged 5 commits intopantsbuild:mainfrom
sureshjoshi:14190-js-ts-support

Conversation

@sureshjoshi
Copy link
Copy Markdown
Member

@sureshjoshi sureshjoshi commented May 13, 2022

This PR provides step forward for a Javascript backend. It implements the target types js_source and js_sources, as well as tailor support. Creating Javascript test targets is deferred to a later PR.

Typescript support is intentionally excluded at this time.

(See #14190)

[ci skip-rust]
[ci skip-build-wheels]

# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
@sureshjoshi
Copy link
Copy Markdown
Member Author

This base support was honestly a copy/paste/find/replace of @tdyas cc code.

I left dependency_inference out of this, until I can better understand how that works - I'm PR'ing this code, so that I can add the NodeJS tool and PrettierJS functionality from my personal plugins repo.

@sureshjoshi sureshjoshi added the category:internal CI, fixes for not-yet-released features, etc. label May 13, 2022
Copy link
Copy Markdown
Contributor

@Eric-Arellano Eric-Arellano left a comment

Choose a reason for hiding this comment

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

Yay!

Comment thread src/python/pants/backend/js/target_types.py Outdated
Comment thread src/python/pants/backend/js/target_types.py
Comment thread src/python/pants/backend/experimental/js/BUILD
Comment thread src/python/pants/backend/js/target_types.py
# Rust tests and lints will be skipped. Delete if not intended.
[ci skip-rust]

# Building wheels and fs_util will be skipped. Delete if not intended.
[ci skip-build-wheels]
@Eric-Arellano
Copy link
Copy Markdown
Contributor

Ready for me to merge once CI goes green?

@sureshjoshi
Copy link
Copy Markdown
Member Author

Yep!

@sureshjoshi
Copy link
Copy Markdown
Member Author

Associated GH ticket stays open though, still have 2-3 more PRs

@Eric-Arellano Eric-Arellano enabled auto-merge (squash) May 13, 2022 17:52
@Eric-Arellano Eric-Arellano merged commit 08e3879 into pantsbuild:main May 13, 2022
@sureshjoshi sureshjoshi deleted the 14190-js-ts-support branch May 13, 2022 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:internal CI, fixes for not-yet-released features, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants