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

Test PRs are added to the bottom #173

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Test PRs are added to the bottom #173

wants to merge 5 commits into from

Conversation

javierm
Copy link
Owner

@javierm javierm commented Aug 3, 2022

No description provided.

@@ -1,6 +1,6 @@
<%= form_tag(budget_executions_path(budget), method: :get, class: "budget-executions-filters") do %>
<div class="filter">
<%= label_tag :milestone_tag, t("budgets.executions.filters.milestone_tag.label") %>
<%= label_tag :milestone_tag,t("budgets.executions.filters.milestone_tag.label") %>
Copy link

Choose a reason for hiding this comment

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

Layout/SpaceAfterComma: Space missing after comma. (https://rubystyle.guide#spaces-operators)

@@ -1,7 +1,7 @@
(function() {
"use strict";
App.LegislationAnnotatable = {
makeEditableAndHighlight: function(colour) {
makeEditableAndHighlight: function( colour) {
Copy link

Choose a reason for hiding this comment

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

There should be no spaces inside this paren.

@@ -3,6 +3,7 @@ class Administrator
include CanCan::Ability

def initialize(user)

Copy link

Choose a reason for hiding this comment

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

Layout/EmptyLinesAroundMethodBody: Extra empty line detected at method body beginning. (https://rubystyle.guide#empty-lines-around-bodies)

@@ -24,7 +24,8 @@
height: rem-calc(14);
position: relative;

.meter {
.meter
{
Copy link

Choose a reason for hiding this comment

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

Opening curly brace { should be preceded by one space

@javierm javierm added this to Reviewing in Automation test via automation Aug 22, 2022
@javierm javierm removed this from Reviewing in Automation test Aug 22, 2022
@javierm javierm closed this Aug 22, 2022
@javierm javierm reopened this Aug 22, 2022
@javierm javierm added this to Reviewing in Automation test Aug 22, 2022
@javierm javierm changed the title Test new permissions in github actions Test PRs are added to the bottom Aug 22, 2022
@javierm javierm removed this from Reviewing in Automation test Aug 22, 2022
@javierm javierm closed this Aug 22, 2022
@javierm javierm reopened this Aug 22, 2022
@javierm javierm added this to Reviewing in Automation test Aug 22, 2022
@javierm javierm closed this Aug 22, 2022
@javierm javierm removed this from Reviewing in Automation test Aug 22, 2022
@javierm javierm reopened this Aug 22, 2022
@javierm javierm added this to Reviewing in Automation test Aug 22, 2022
@javierm javierm removed this from Reviewing in Automation test Aug 22, 2022
@javierm javierm closed this Aug 22, 2022
@javierm javierm reopened this Aug 22, 2022
@javierm javierm added this to Reviewing in Automation test Aug 22, 2022
@javierm javierm moved this from Reviewing to Testing in Automation test Aug 22, 2022
@javierm javierm removed this from Testing in Automation test Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
1 participant