Skip to content
This repository has been archived by the owner on Sep 15, 2021. It is now read-only.

Support platform_common and ToolchainInfo #100

Closed
leostera opened this issue Jun 21, 2018 · 2 comments
Closed

Support platform_common and ToolchainInfo #100

leostera opened this issue Jun 21, 2018 · 2 comments

Comments

@leostera
Copy link

Hi!

I'm trying to use skydoc in some rules I'm writing for ReasonML but I'm having trouble with platform_common being undefined.

  File "external/com_github_ostera_rules_reason/reason/private/bs_module.bzl", line 127, in <module>
    providers = [platform_common.ToolchainInfo],
NameError: name 'platform_common' is not defined

From looking around I saw the fix for provider and I think I can add to the rule_extractor.py another stub for platform_common so that I can generate docs.

What do you think? :)

@laurentlb
Copy link
Contributor

Yes, feel free to add more stubs.
We know it's brittle, so Skydoc is going to be rewritten in a much more robust way (cc @c-parsons).

@c-parsons
Copy link
Contributor

Closing. Use Stardoc :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants