tree 801759bdfea49760eeed60a4ad6de778077337ce
parent 636430a30f3eef86062640d66e049df37ad16711
author adonovan <adonovan@google.com> 1581548283 -0800
committer Copybara-Service <copybara-worker@google.com> 1581548341 -0800

bazel syntax: avoid StarlarkThread.getGlobals within lib.syntax

Another baby step to decoupling these classes: this change exposes a
Module parameter from various lib.syntax API functions instead of
using thread.getGlobals(), in most cases pushing the call into the
callers.

Construction of the thread and module is still entangled so we can't
do more yet.

Also:
- rename parseAndValidate{,Skylark} and
  replace thread by semantics parameter.
  Now ValidatorTest doesn't depend on Thread.

This is a breaking API change for copybara.

PiperOrigin-RevId: 294764678