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

Ember build continually rebuilds in response to dist directory changes #130

Open
NullVoxPopuli opened this issue Jul 11, 2020 · 2 comments

Comments

@NullVoxPopuli
Copy link
Contributor

NullVoxPopuli commented Jul 11, 2020

The repeated messages

file changed dist/workers/ai__launcher.js
⠋ building... '@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
'@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used
⠙ building... '@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
'@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used
[project root]/workers/ai.js
import Worker, { method } from '@skyrocketjs/worker';

import { run as runRNN } from './ai/rnn';
import { run as runRandom } from './ai/random';

export default class AIWorker extends Worker {
  @method
  run({ game, algorithm, trainingData}) {
    switch (algorithm) {
      case 'RNN':
        return runRNN(game, trainingData);
      case 'random':
        return runRandom();
      default:
        console.error(...arguments);
        throw new Error('Unrecognized Algorithm', algorithm);
    }
  }
}
Full output
 building... The 'this' keyword is equivalent to 'undefined' at the top level of an ES module, and has been rewritten
⠇ building... [broccoli-persistent-filter:Babel > applyPatches]'@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
'@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used
⠏ building... [broccoli-persistent-filter:Babel > applyPatches]'@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
'@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used

Build successful (3591ms) – Serving on http://localhost:4200/

Slowest Nodes (totalTime >= 5%)                                                                                             | Total (avg)                   
----------------------------------------------------------------------------------------------------------------------------+-------------------------------
broccoli-persistent-filter:Babel (4)                                                                                        | 1209ms (302 ms)               
Babel: doctor-who-ai (2)                                                                                                    | 809ms (404 ms)                
Bundler (1)                                                                                                                 | 374ms                         
ember-auto-import-analyzer (3)                                                                                              | 288ms (96 ms)                 
Babel: @ember/test-helpers (2)                                                                                              | 227ms (113 ms)                

file added dist
⠋ building... '@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
'@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used
'@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
⠙ building... '@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used

Build successful (155ms) – Serving on http://localhost:4200/

Slowest Nodes (totalTime >= 5%)                                                                                             | Total (avg)                   
----------------------------------------------------------------------------------------------------------------------------+-------------------------------
WorkerCompiler extends BroccoliRollup (2)                                                                                   | 44ms (22 ms)                  
BroccoliMergeTrees (20)                                                                                                     | 14ms (0 ms)                   
broccoli-persistent-filter:Babel (4)                                                                                        | 11ms (2 ms)                   

file changed dist/workers/ai__launcher.js
⠋ building... '@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
'@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used
⠙ building... '@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
'@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used

Build successful (209ms) – Serving on http://localhost:4200/

Slowest Nodes (totalTime >= 5%)                                                                                             | Total (avg)                   
----------------------------------------------------------------------------------------------------------------------------+-------------------------------
WorkerCompiler extends BroccoliRollup (2)                                                                                   | 75ms (37 ms)                  
BroccoliMergeTrees (20)                                                                                                     | 14ms (0 ms)                   
broccoli-persistent-filter:Babel (4)                                                                                        | 13ms (3 ms)                   

file changed dist/workers/ai__launcher.js
⠋ building... '@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
'@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used
'@skyrocketjs/worker/build/shell' is imported by workers/ai--launcher.js, but could not be resolved – treating it as an external dependency
⠙ building... '@skyrocketjs/worker' is imported by workers/ai.js, but could not be resolved – treating it as an external dependency
'method' is imported from external module '@skyrocketjs/worker' but never used

Build successful (175ms) – Serving on http://localhost:4200/
@ming-codes
Copy link

ming-codes commented Oct 3, 2020

I'm getting this too, but it looks like it's fixed on master?

@ming-codes
Copy link

I made a few changes on my local. It looks like skyrocket is mostly usable. This issue is a major blocker. Maybe we can cut a release?

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

No branches or pull requests

2 participants