Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Jan 18 19:42
    claassistantio commented #484
  • Jan 18 19:42
    dependabot-preview[bot] labeled #484
  • Jan 18 19:42
    dependabot-preview[bot] opened #484
  • Jan 18 19:42

    dependabot-preview[bot] on nuget

    Bump YamlDotNet from 8.0.0 to 8… (compare)

  • Jan 18 01:05
  • Jan 17 23:05
    claassistantio commented #483
  • Jan 17 23:04
    dependabot-preview[bot] labeled #483
  • Jan 17 23:04
    dependabot-preview[bot] opened #483
  • Jan 17 23:04

    dependabot-preview[bot] on nuget

    Bump Stripe.net from 34.15.0 to… (compare)

  • Jan 17 14:45
    ChiefLegend commented #477
  • Jan 17 14:17
    niemyjski commented #477
  • Jan 17 03:18

    niemyjski on master

    Updates to latest stable founda… (compare)

  • Jan 17 03:06

    dependabot-preview[bot] on nuget

    (compare)

  • Jan 17 03:06

    niemyjski on master

    Bump Microsoft.Extensions.Loggi… Merge pull request #481 from ex… (compare)

  • Jan 17 03:06
    niemyjski closed #481
  • Jan 17 03:06
    dependabot-preview[bot] synchronize #481
  • Jan 17 03:06

    dependabot-preview[bot] on nuget

    Bump Microsoft.Extensions.Loggi… (compare)

  • Jan 17 03:06
    dependabot-preview[bot] edited #481
  • Jan 17 03:05
    claassistantio commented #479
  • Jan 17 03:05
    dependabot-preview[bot] edited #481
Blake Niemyjski
@niemyjski
awe sweet
Frank Ebersoll
@frankebersoll
there's one problem: it will be much more fun if the build itself is faster.
Blake Niemyjski
@niemyjski
merge immediately :)
Frank Ebersoll
@frankebersoll
and we have no incremental builds yet - i don't know if tsproject supports them.
Blake Niemyjski
@niemyjski
the thing with 36, we’ve made some commits since then, we gotta make sure it merges properly.
Frank Ebersoll
@frankebersoll
i mean, it has to spit out the whole thing every time.
Blake Niemyjski
@niemyjski
tsproject does I think
if it doesn’t we could create an issue
that guy is really good about it
Frank Ebersoll
@frankebersoll
my question is, do we really need it during development time?
build, watch and watch-test should do as little work as possible.
Blake Niemyjski
@niemyjski
no, I think we are fine
although, do we have to transpile for tests?
I think that’s a yes
Frank Ebersoll
@frankebersoll
we need an additional publish task that does packaging, final tests and then pushes it
at least, that's what i imagine right now
yes, we need to transpile, and with source maps so.
because it will make debugging much easier. i've enabled source maps in karma, so we get stack traces with actual typescript references
no more searching in js files
it also enables stepping into typescript. i didn't get that with our current solution, sadly
Blake Niemyjski
@niemyjski
yeah that will be really nice
well we have the build step
which publish relies on
so we should be fine. unless all of that changed
Frank Ebersoll
@frankebersoll
build takes about 20 seconds on my machine now
with gulp-tsb and incremental builds i got it down to 0.5 seconds when a file changes.
Blake Niemyjski
@niemyjski
well we don’t have to run the linter
that should make it faster no?
Frank Ebersoll
@frankebersoll
so, build could be only what is needed to run and debug stuff. and publish would do minifying, packaging and whatnot
yes, but even that could be included in watch
and even before building, because, i don't want to build if my source code has errors
i'll try to get incremental builds working with tsproject
but first, i fix stringify
#36 has been rebased onto all commits that have happened in the meantime
should be ready for merge
Blake Niemyjski
@niemyjski
ok
did you sync?
Frank Ebersoll
@frankebersoll
?
Blake Niemyjski
@niemyjski
push your changes I dont’ see the rebase or I guess it doesn’t show up in github?
Frank Ebersoll
@frankebersoll
i have rewritten the changes so they include your commits
Blake Niemyjski
@niemyjski
ok, do you think we should merge it now or after our release?
it’s your call
if it’s going to make our lives easier lets do it now
Frank Ebersoll
@frankebersoll
this should be non-breaking, as a recompile lead to no changes in dist
Blake Niemyjski
@niemyjski
ok
Frank Ebersoll
@frankebersoll
only things to change in dist have been maps, which makes sense
Blake Niemyjski
@niemyjski
ok
merged
give me a few minutes