Charcoal HQ

Undo

8:00 PM

Oh, found it

> Webpacker requires Node.js >= v6.4 and you are using v4.2.6

When running it manually

J F

I’d recommend 7.

Undo

k

Looks like 6.11 is LTS, 8.1 is current? nodejs.org/en

J F

So 8 then. Misread.

Undo

k

Configuring Cap to respect nvm

J F

Is there a specific range of browsers we target with MS? I’m currently using this query, but I can change it to anything.

Undo

8:07 PM

I suspect most everyone here is using something modern-ish

J F

Do you have any analytics?

Undo

Should, gimme a sec

Heh, so I have server logs but nothing saying what site it's for

Shows you how much I use nginx logs

also, what the heck is Yandex and why is it indexing metasmoke?

Russian search engine

It needs to learn how to throttle

DavidPostill

@Undo lol. block it in robots.txt

Undo

metasmoke.erwaysoftware.com/robots.txt

SmokeDetector

[ SmokeDetector | MS ] Few unique characters in answer, no whitespace in answer, repeating characters in answer: Printrun doesn't connect when 12 V is on by user7427 on 3dprinting.SE

DavidPostill

8:16 PM

@Undo Hmm.

> The Yandex robot supports the robot exclusion standard with the enhanced capabilities that are described below.

yandex.com/support/webmaster/controlling-robot/robots-txt.xml

Undo

I disagree :P

DavidPostill

Clearly all lies :)

J F

Time for a 402 Payment Required?

Undo

I was thinking a 301 to one of those ten-hour YouTube videos

Wow, compiling Webpack assets is a pretty heavy operation

Is this supposed to eat 1G+ RAM?

J F

@Undo How long has it taken so far?

Undo

8:22 PM

couple minutes

Dev compilation is fast; prod is slow

ms down temp

I've seen this before with first-run asset compilation, though

J F

I’ve bundled jQuery and Bootstrap in application.js. Should I get them from a CDN instead?

Undo

Might be a good idea

Runs in a couple seconds without RAILS_ENV=production

SmokeDetector

Restart: API quota is 13380.

Undo

@JF Yeah, probably a good idea to pull them from a CDN. I'm not having luck compiling them

Which is kinda weird

Also having to run yarn after every load

J F

8:38 PM

@Undo Node dependencies go in ./node_modules, so that might get wiped on deploy.

Undo

kk, we'll persist that

SmokeDetector

[ metasmoke ] ci/circleci success on 1d900ce: Your tests passed on CircleCI!

Undo

Still trying to eat all the RAM in prod mode

Finished eventually, about an order of magnitude longer than dev mode

J F

Does changing config/webpack/production.js:11 to devtool: false, make it faster?

ArtOfCode

...why have we got Node dependencies in a Rails project?

J F

8:44 PM

@ArtOfCode ES6!!1

ArtOfCode

that... doesn't help

Michał Perłakowski

@JF You should use tree shaking in webpack, so the bundle will include only the code that is actually used.

J F

@MichałPerłakowski That’s enabled by default AFAIK. Unfortunately, most libraries don’t have ES6-module versions yet (they’re required for tree shaking).

@ArtOfCode Because webpack is Node-based, and that’s the future of client-side JS.

Michał Perłakowski

@JF I would say "present of client-side JS".

Undo

Don't know what's going on here - Cap failing on asset precompile:

02:19 deploy:assets:precompile
      01 ls /var/railsapps/metasmoke/releases/20170610204030/assets_manifest_backup
      01 ls:
      01 cannot access '/var/railsapps/metasmoke/releases/20170610204030/assets_manifest_backup'
      01 : No such file or directory
      01
      02 ~/.rvm/bin/rvm 2.4.1@metasmoke do bundle exec rake assets:precompile
      02 /var/railsapps/metasmoke/releases/20170610204524/config/initializers/appconfig.rb:3: warning: already initialized constant AppConfig
      02 /var/railsapps/metasmoke/releases/20170610204524/config/environments/production.rb:74: warning: p…

But I can go to the release directory and run ~/.rvm/bin/rvm 2.4.1@metasmoke do bundle exec rake assets:precompile rails_env=PRODUCTION just fine

J F

8:48 PM

Also, babili, which minifies ES6+ stuff. Babel transpiles it all down to stuff that browsers can run today.

Undo

Might be because Capistrano isn't respecting nvm

J F

The PrecompileRequired bit comes from capistrano-plugins/capistrano-faster-assets, which raises it because it can’t find the ./assets_manifest_backup directory.

Undo

Let's try turning it off

J F

It seems like it checks for that directory, and runs rake assets:precompile if it isn’t present.

Undo

Right, so shouldn't make a difference here

J F

8:58 PM

rake assets:precompile runs rake webpacker:compile, which should print “Webpacker is installed 🎉 🍰” or “Configuration config/webpacker.yml file not found”.

Undo

I get the cake and stuff when I run it manually. Pretty sure it's nvm, chasing it

There we go. Looks like it was an NPM issue

J F

\o/

Undo

9:26 PM

→ 5 messages moved to Trashcan

Merged and on prod @JF, looks like it's working well

Thank you so much for figuring this out :)

SmokeDetector

[ SmokeDetector | MS ] Bad keyword with email in answer, email in answer: How to balance debts and enjoying life? by Ambler Shannon on money.SE

Undo

@ArtOfCode @Henders FYI, foreman start to run metasmoke in dev now.

SmokeDetector

tpu- by Gothdo

ArtOfCode

@Undo and prod?

Undo

Works the same as before

(meaning "Capistrano takes care of it")

Everything's precompiled in prod, the difference is that dev lets you edit things on the fly

ArtOfCode

9:35 PM

@Undo but prod locally

still running rails s -e production for that locally, or some variant of foreman?

Undo

@ArtOfCode Should be able to run export RAILS_ENV=production; rails assets:precompile; rails s

So same as before, IIRC

J F

@Undo Except for the JS :(

!!/alive

SmokeDetector

@JF You doubt me?

Undo

@JF Huh, worked for me before, doesn't now. Caching?

J F

@Undo probably.

SmokeDetector

9:46 PM

[ metasmoke ] ci/circleci failure on 65f53b8: Your tests failed on CircleCI

Undo

NPM tests

J F

Just warnings: import/no-unassigned-import

Undo

yep

faster-assets needs more tuning, too

SmokeDetector

[ metasmoke ] ci/circleci failure on f162264: Your tests failed on CircleCI

Undo

Looks like the only thing now is that we lost timestamps @JF

quartata

9:51 PM

@ArtOfCode oh no

not metasmoke

J F

oh yes :evil_grin:

quartata

nuuuuu

J F

@Undo Somehow vendor.js isn’t getting added in the <head>.

Undo

Should it be in the filesystem somewhere?

Shows that it's being created

J F

It gets generated by app/javascript/packs/vendor.js and is supposed to be included at app/views/layouts/application.html.erb:21.

Undo

9:54 PM

vendor-45c5d0fb302763eaccac.js is in public/packs

Oh

<% doesn't render anything. <%= does

J F

headdesk

Undo

I'm fixing

best commit message ever

J F

lol

Undo

Timestamps work

SmokeDetector

[ metasmoke ] ci/circleci failure on e7589f3: Your tests failed on CircleCI

Undo

9:58 PM

oh, here's one. @JF looks like something happened to the Rails default helper JS? I've never had to touch it and I don't even know what it's called.

See https://metasmoke.erwaysoftware.com/flagging/conditions

<a data-confirm="Are you sure?" class="text-danger" rel="nofollow" data-method="delete" href="/flagging/conditions/5">Delete</a>

^ Should be intercepted by some kind of Rails JS somewhere to send a DELETE request instead of a GET

J F

It’s jquery-ujs. I’ll make a PR with a couple of other improvements in a few.

SmokeDetector

10:16 PM

API quota rolled over with 12708 requests remaining. Current quota: 19999.

math: 220
stackoverflow: 83
superuser: 31
askubuntu: 21
physics: 14
english: 12
android: 11
mathoverflow.net: 10
tex: 10
unix: 10
serverfault: 9
electronics: 9
codegolf: 9
workplace: 9
apple: 8
gaming: 7
security: 7
skeptics: 7
islam: 7
academia: 6
diy: 6
worldbuilding: 6
arduino: 6
wordpress: 5
drupal: 5
ell: 5
photo: 5
stats: 5
es.stackoverflow: 5
money: 4
graphicdesign: 4
movies: 4
ru.stackoverflow: 4
mathematica: 4
robotics: 4
dsp: 3
dba: 3
webapps: 3
cogsci: 3
bitcoin: 3
rpg: 3
politics: 3

ArtOfCode

@Undo jquery-ujs

unobtrusive JS drivers

J F

23 mins ago, by J F

It’s jquery-ujs. I’ll make a PR with a couple of other improvements in a few.

SmokeDetector

[ metasmoke ] ci/circleci failure on 4771de5: Your tests failed on CircleCI

J F

try the next commit, Circle

SmokeDetector

[ metasmoke ] ci/circleci success on 9b2f31a: Your tests passed on CircleCI!

J F

10:26 PM

sd 3poof

Undo

Huh, now I can't compile it

J F

Any errors?

SmokeDetector

[ metasmoke ] ci/circleci success on 845e2d2: Your tests passed on CircleCI!

Undo

nvm, need to run yarn

Yep, works now. Thanks!

J F

np

Zachiel

10:59 PM

Hello! I have noticed an attempt at spam of a kind that I don't think was reported: someone tried to sneak a telephone number as "edit reason" in a post on RPG.SE (I noticed and marked the edit review as spammy, but some other people just marked it as "not improving the answer".)

J F

@Zachiel Have a link?

Zachiel

Unfortunately, I can't really remember what the answer was about.

J F

This one?

Zachiel

It looks like this, yes. Maybe I assumed it didnìt get reported because I thought FireAlert was the service for detecting spam (and we get Fire Alert notifies in chat) but I saw no such notify.

DavidPostill

@Zachiel Hmm. Spamming in edit reasons would be weird. However 555-555-0199 is a fictional phone number

> In fact, only 555-0100 through 555-0199 are now specifically reserved for fictional use

So it's just a stupid edit reason and not actually spam.

Zachiel

11:08 PM

@DavidPostill good to know

doppelgreener

i have, actually, seen legit attempted spam in edit reasons before once or twice.

it's weird but hey i guess it's a text box, so they'll dump the thing in there

DavidPostill

Edit reasons is not somewhere where Smokey (or I assume Fire Alert) checks for spam though.

SmokeDetector

[ SmokeDetector | MS ] Link at end of answer: sudo: npm: command not found by spreadingideas on stackoverflow.com (@quartata)

doppelgreener

11:25 PM

@DavidPostill oh good

wait

oh dang okay

i misread at first. i guess it's mostly tucked away there though, so people are hardly going to notice.

J F

11:42 PM

PSA: If Undo deploys metasmoke and you have a page open, it’ll ~~order~~ politely ask you to reload the page.

Also, if you are offline and come back online and Undo has deployed in the interim, it’ll do the same thing.

Transcript for