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

#1502 webpack (connect #1502) #3000

Merged
merged 42 commits into from
Mar 29, 2019
Merged

#1502 webpack (connect #1502) #3000

merged 42 commits into from
Mar 29, 2019

Conversation

finnfiddle
Copy link
Contributor

@finnfiddle finnfiddle commented Mar 7, 2019

Before the PR (what is the issue or what needed to be done)

The solution

Screenshots (if appropriate)

Checklist

  • Connect the issue
  • Test plan
  • Copyright header
  • Code formatting
  • Documentation

@ghost ghost assigned finnfiddle Mar 7, 2019
@ghost ghost added the In progress label Mar 7, 2019
@ghost ghost assigned kymni Mar 12, 2019
@janagombitova janagombitova changed the title #1502 webpack #1502 webpack (connect #1502) Mar 14, 2019
@kymni kymni requested a review from muloem March 18, 2019 08:24
@@ -0,0 +1,83 @@
{
"extends": "airbnb",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is based on some styling from airbnb? I guess its relatively stable right? i.e. we wont have broken checks if airbnb decides to refurbish?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes this is one of the most commonly used styleguides - if not THE most used. no need to worry

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also the version is locked so it wont just suddenly break

@@ -17,115 +17,6 @@ class LoaderFilter < WebFilters::MinispadeFilter
end
end

class EmberAssertFilter < Filter
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You probably need to cleanup AssetfilePublic the same way, right?

Copy link
Contributor Author

@finnfiddle finnfiddle Mar 29, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes youre right. we can do this at a later stage though because the hybrid build is still getting the job done as it is

* develop: (48 commits)
  [#3026]Stop updating the backend.
  [#3024] release notes
  Update bootstrap-deploy.sh
  [#3022]Avoid NPE if no user found.
  [#3018] release notes
  [#3018]Initial version of release notes.
  [#2969] Remove whitespace
  [#2971]Extract constants. Remove trailing whitespace.
  [#3017] Add NSS package
  [#3017] Add NSS package
  [#3017] Add NSS package
  [#3017] Add NSS package
  [#2970]Remove trailing whitespace.
  [#2970]Formatting...
  [#2802] unused function
  [#2802] do it the ember way
  [#2694] Remove backends configuration file
  [#2696] Refactor cascade node deletion
  [#2694] Remove backends deployment code
  [#2971]Stop calling taks on backend. Remove unused stuff.
  ...
@muloem muloem merged commit e0b1a08 into develop Mar 29, 2019
@muloem muloem deleted the #1502-webpack branch March 29, 2019 10:55
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

Successfully merging this pull request may close these issues.

4 participants