uber-web / jazelle Goto Github PK

View Code? Open in Web Editor NEW

100.0 100.0 9.0 20.5 MB

Incremental, cacheable builds for large Javascript monorepos using Bazel

License: MIT License

JavaScript 91.71% Starlark 5.46% Shell 2.75% Makefile 0.04% Go 0.04%

jazelle's People

Contributors

Stargazers

Watchers

Forkers

denyska yonet dianasuvorova shykisto filmaven mike-kaufman pdufour sbalabanov nnunley

jazelle's Issues

`jz outdated --json` emits invalid json

running jz outdated --json emits output that is not valid json.

In particular,

the list of entries are not wrapped in [ and ] at the head and tail of the outpu
the list of entries are not delimited by commas.

See code here:

jazelle/commands/outdated.js

Lines 160 to 182 in c2e9658

    
           for (const result of results) { 
        
             const formatted = []; 
        
             if (dedup) { 
        
               formatted.push(result); 
        
             } else { 
        
               result.installed.forEach(version => 
        
                 formatted.push({ 
        
                   ...result, 
        
                   installed: [version], 
        
                 }) 
        
               ); 
        
             } 
        
             if (json) logger('['); 
        
             formatted.forEach((entry, i) => 
        
               json 
        
                 ? logger( 
        
                     JSON.stringify(entry) + (i !== formatted.length - 1 ? ',' : '') 
        
                   ) 
        
                 : logger(entry.packageName, entry.installed.join(' '), entry.latest) 
        
             ); 
        
             if (json) logger(']'); 
        
           }

installation with yarn2.

Readme needs to be updated that with yarn2 correct command is yarn dlx jazelle not yarn global add jazelle
yarn dlx jazelle throws an error while appears to be working correctly:

yarn dlx jazelle
➤ YN0000: ┌ Resolution step
➤ YN0000: └ Completed in 1s 654ms
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed in 0s 222ms
➤ YN0000: ┌ Generating workspace-deps.txt
➤ YN0000: └ Completed
➤ YN0000: Done in 1s 926ms

/private/var/folders/fz/tds93qq164s2q0sv2zgbd7k40000gn/T/xfs-24c61363/dlx-30285/.yarn/unplugged/jazelle-npm-0.0.0-standalone.32-canary-0ecae71df3/node_modules/jazelle/bin/bootstrap.sh:3
realpath() {
           ^

SyntaxError: Unexpected token '{'
    at wrapSafe (internal/modules/cjs/loader.js:915:16)
    at Module._compile (internal/modules/cjs/loader.js:963:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.external_module_.Module._load (/private/var/folders/fz/tds93qq164s2q0sv2zgbd7k40000gn/T/xfs-24c61363/dlx-30285/.pnp.js:4902:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)
    at internal/main/run_main_module.js:17:47

Installation in a polyglot repo?

Hi, love this project idea and cool to see it develop!

I have a monorepo with a setup of

WORKSPACE
BUILD
java-package-1
java-2
... (dozens more)
frontend

BUILD
package.json
yarn.lock

where frontend code is all in a subfolder, there is no package.json or yarn lock in the root of the monorepo. From what I can tell, Jazelle expects those files to be in the root. Is there a way of overriding this?

jazelle upgrade does not upgrade peerDependencies

Currently only targets dependencies and devDependencies. Should also target peerDependencies.

New repo fails on M1 mac

Trying to run jz lint throws the error:

ERROR: /Users/christopherthomas/github/xiot/jazelle-test/WORKSPACE:9:21: fetching _jazelle_dependencies rule //external:jazelle_dependencies: Traceback (most recent call last):
        File "/private/var/tmp/_bazel_christopherthomas/fef563f5da3b0b72a80e398e2ba88e0c/external/jazelle/rules/jazelle-dependencies.bzl", line 50, column 25, in _jazelle_dependencies_impl
                sha256 = node_sha256[binary],
Error: key "darwin-arm64" not found in dictionary

It looks like like darwin-arm64 is missing from the jazelle_dependencies/node_sha256 in WORKSPACE

uber-web / jazelle Goto Github PK

jazelle's People

Contributors

Stargazers

Watchers

Forkers

jazelle's Issues

`jz outdated --json` emits invalid json

installation with yarn2.

Please add license

Installation in a polyglot repo?

jazelle upgrade does not upgrade peerDependencies

New repo fails on M1 mac

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	for (const result of results) {
	const formatted = [];
	if (dedup) {
	formatted.push(result);
	} else {
	result.installed.forEach(version =>
	formatted.push({
	...result,
	installed: [version],
	})
	);
	}

	if (json) logger('[');
	formatted.forEach((entry, i) =>
	json
	? logger(
	JSON.stringify(entry) + (i !== formatted.length - 1 ? ',' : '')
	)
	: logger(entry.packageName, entry.installed.join(' '), entry.latest)
	);
	if (json) logger(']');
	}