chore: removing template-inline-precompile

[kushthedude]

ember-cli-htmlbars-inline-precompile is not needed anymore with dependencies upgrades.

[iamareebjamal]

@kushthedude No need to merge dev into branches. You're just wasting CI minutes that way, a new build with no changes will be created. The only need for merges in when there are conflicts, DB migrations or interdependent dependencies

[kushthedude]

@kushthedude No need to merge dev into branches. You're just wasting CI minutes that way, a new build with no changes will be created. The only need for merges in when there are conflicts, DB migrations or interdependent dependencies

Then why there is need for update branch button, Also isn't target branch needs to be in sync with HEAD branch before development in this option ?

[iamareebjamal]

Then why there is need for update branch button

Because of this - The only need for merges in when there are conflicts, DB migrations or interdependent dependencies

There's an option to merge without CI passing, should that be done as well? I wish there was an option to restrict branch updates to only admin

Also isn't target branch needs to be in sync with HEAD branch before development in this option ?

No

[kushthedude]

No

So you can merge a PR whose head is not in sync with development 😕 ?

[iamareebjamal]

Yes

[codecov]

Codecov Report

Merging #3943 into development will not change coverage.
The diff coverage is 60%.

@@             Coverage Diff              @@
##           development    #3943   +/-   ##
============================================
  Coverage        21.98%   21.98%           
============================================
  Files              460      460           
  Lines             4727     4727           
============================================
  Hits              1039     1039           
  Misses            3688     3688

Impacted Files	Coverage Δ
app/controllers/register.js	27.58% <ø> (ø)	⬆️
app/controllers/oauth/callback.js	0% <ø> (ø)	⬆️
app/components/account/application-section.js	0% <ø> (ø)	⬆️
app/controllers/public/index.js	0% <0%> (ø)	⬆️
app/components/forms/login-form.js	62.06% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d4b36a4...6952048. Read the comment docs.

[kushthedude]

Might be an admin only feature then😅

…

On Sat, 1 Feb, 2020, 12:11 codecov[bot], ***@***.***> wrote: Codecov <https://codecov.io/gh/fossasia/open-event-frontend/pull/3943?src=pr&el=h1> Report Merging #3943 <https://codecov.io/gh/fossasia/open-event-frontend/pull/3943?src=pr&el=desc> into development <https://codecov.io/gh/fossasia/open-event-frontend/commit/d4b36a44057befcd2c6bbbe43eaea59f68417f9d?src=pr&el=desc> will *not change* coverage. The diff coverage is n/a. [image: Impacted file tree graph] <https://codecov.io/gh/fossasia/open-event-frontend/pull/3943?src=pr&el=tree> @@ Coverage Diff @@ ## development #3943 +/- ## ============================================ Coverage 21.98% 21.98% ============================================ Files 460 460 Lines 4727 4727 ============================================ Hits 1039 1039 Misses 3688 3688 ------------------------------ Continue to review full report at Codecov <https://codecov.io/gh/fossasia/open-event-frontend/pull/3943?src=pr&el=continue> . *Legend* - Click here to learn more <https://docs.codecov.io/docs/codecov-delta> Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov <https://codecov.io/gh/fossasia/open-event-frontend/pull/3943?src=pr&el=footer>. Last update d4b36a4...6952048 <https://codecov.io/gh/fossasia/open-event-frontend/pull/3943?src=pr&el=lastupdated>. Read the comment docs <https://docs.codecov.io/docs/pull-request-comments> . — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3943?email_source=notifications&email_token=AKQMTLQJ3GAHZKTDXJWUPBTRAUKQ7A5CNFSM4KORI7SKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKQV4IQ#issuecomment-581000738>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AKQMTLQBOLMYGJMBFAGSSQTRAUKQ7ANCNFSM4KORI7SA> .