Commit Graph

42 Commits

Author SHA1 Message Date
Rafael Mendonça França
fe1f4b2ad5
Add more rubocop rules about whitespaces 2016-10-29 01:17:49 -02:00
Xavier Noria
bb1ecdcc67 fixes remaining RuboCop issues [Vipul A M, Xavier Noria] 2016-09-01 23:41:49 +02:00
Rafael Mendonça França
55f9b8129a
Add three new rubocop rules
Style/SpaceBeforeBlockBraces
Style/SpaceInsideBlockBraces
Style/SpaceInsideHashLiteralBraces

Fix all violations in the repository.
2016-08-16 04:30:11 -03:00
Xavier Noria
a9dc45459a code gardening: removes redundant selfs
A few have been left for aesthetic reasons, but have made a pass
and removed most of them.

Note that if the method `foo` returns an array, `foo << 1`
is a regular push, nothing to do with assignments, so
no self required.
2016-08-08 01:12:38 +02:00
Xavier Noria
80e66cc4d9 normalizes indentation and whitespace across the project 2016-08-06 20:16:27 +02:00
Xavier Noria
92e2d16a3c applies new string literal convention in tools
The current code base is not uniform. After some discussion,
we have chosen to go with double quotes by default.
2016-08-06 19:22:51 +02:00
Prathamesh Sonpatki
ea491653cc Remove requiring load_paths from tools/test.rb
- Instead require and setup Bundler
- `tools/test.rb` is used internally from all `bin/test` scripts inside
   component gems.
- Followup of 2abcdfd978.
2016-03-02 10:28:34 +05:30
Arthur Neves
2abcdfd978 Remove load_paths file 2016-02-27 13:03:57 -05:00
Aaron Ang
5bd179787f Fix title of README according to Markdown conventions
The first heading in some README's are indicated using a second level
heading (`##`), which in my opinion is of incorrect structure.
Therefore, in this patch I changed the first heading to a first level
heading (`#`) in README's where this incorrect structure occurs.

[ci skip]
2016-02-25 03:39:02 +01:00
Sourav Moitra
2f638b227f [ci skip] line_statistics added to README 2016-02-14 21:40:13 +05:30
Yves Senn
2e59604909 make it possible to customize the executable inside rereun snippets.
In the Rails repository we use a `bin/test` executable to run our tests.
However the rerun snippets still included `bin/rails test`:

BEFORE:
```
Failed tests:

bin/rails test test/cases/adapters/postgresql/schema_test.rb:91
```

AFTER:
```
Failed tests:

bin/test test/cases/adapters/postgresql/schema_test.rb:91
```
2015-06-13 11:58:43 +02:00
Yves Senn
9946788775 select the AR adapter through bin/test. 2015-06-11 14:24:56 +02:00
Yves Senn
54d84cbb77 use our runner (bin/test) for framework components.
This adds a script `bin/test` to most Rails framework components. The
script uses the rails minitest plugin to augment the runner.
See https://github.com/rails/rails/pull/19571 for details about the
plugin.

I did not yet add `bin/test` for activerecord, activejob and railties.
These components rely on specific setup performed in the rake-tasks.
2015-06-11 14:12:15 +02:00
schneems
e1a7260640 Use block variable instead of global
```ruby
require 'benchmark/ips'

Benchmark.ips do |x|
  x.report("$&") {
    "foo".sub(/f/) { $&.upcase }
  }
  x.report("block var") {
    "foo".sub(/f/) {|match| match.upcase }
  }
end

```

```
Calculating -------------------------------------
                  $&    48.658k i/100ms
           block var    49.666k i/100ms
-------------------------------------------------
                  $&    873.156k (± 9.3%) i/s -      4.331M
           block var    969.744k (± 9.2%) i/s -      4.818M
```

It's faster, and gets rid of a few "magic" global variables
2015-06-01 19:44:40 -05:00
Sushruth Sivaramakrishnan
6a835e26d3 Doc fix [ci skip] 2015-03-06 23:37:54 +05:30
Rishi Jain
93258c528c minor docs change [ci skip] 2014-11-03 12:37:39 +05:30
Rafael Mendonça França
d02adfaa4d line_statistics is not an executable
Also add documentation on it
2014-10-13 11:07:13 -03:00
Jeremy Kemper
911e7094bd Make a note about the internal tools/ dir and what each utility does 2014-10-13 07:04:26 -07:00
Rafael Mendonça França
4c614651b5 Do not use deprecated exists? method 2014-10-13 10:59:43 -03:00
Rafael Mendonça França
46f80b5094 Document the propose of tools/profile 2014-10-13 10:59:12 -03:00
Rafael Mendonça França
33361a50eb Revert "Merge pull request #17247 from igas/fix-deprecations"
This reverts commit 997f1575f78bf6a2d12b8665a55807d51fe964df, reversing
changes made to f919d5f5ea75f0e56a57643e5a0472367a98625f.

See https://github.com/rails/rails/pull/17247#issuecomment-58895078
2014-10-13 10:55:55 -03:00
Igor Kapkov
9114d65800 remove unused tools/profile 2014-10-13 21:43:20 +08:00
Benjamin Fleischer
91d199259b Encapsulate rake lines from ActiveRecord/ActionPack as CodeTools::LineStatistics
[ci skip]
2014-07-25 13:41:18 -05:00
Benjamin Fleischer
3cbeb8d8ea Fail profiler fast when input is not a ruby file
[ci skip]
2014-07-25 13:41:18 -05:00
Benjamin Fleischer
54a4065074 Optionally add String extensions for ruby-prof printer option
[ci skip]
2014-07-25 13:41:18 -05:00
Benjamin Fleischer
f36c46754f Encapsulate tools/profile functionality in CodeTools::Profiler
Fix bug in path expansion of input filepath

[ci skip]
2014-07-25 13:41:18 -05:00
Benjamin Fleischer
65a227beda Correct tools/profile usage example
[ci skip]
2014-07-25 13:25:42 -05:00
Arun Agrawal
0d062f02e9 Removing Gem.source_index [ci skip] 2013-07-13 12:05:52 +02:00
Jeremy Kemper
00791bfb64 Remove REE GC stats since master is 1.9.3 2012-10-26 08:24:27 -07:00
rochefort
a43213c534 fix Example: Rename tools/profile_requires -> tools/profile 2012-10-06 01:32:02 +09:00
Kornelius Kalnbach
d8c1949684 require "rubygems" is obsolete in Ruby 1.9.3 2012-05-13 14:47:25 +02:00
Prem Sichanugrist
f1637bf2bb Remove Active Resource source files from the repository
Dear Active Resource,

It's not that I hate you or anything, but you didn't get much attention lately. There're so many alternatives out there, and I think people have made their choice to use them than you. I think it's time for you to have a big rest, peacefully in this Git repository.

I will miss you,

@sikachu.
2012-03-13 14:55:44 -04:00
Jeremy Kemper
4883082ff1 Support an extra profile printer arg 2010-06-24 01:08:12 -07:00
Jeremy Kemper
7b730a2f1b Show GC time and # of runs too 2010-06-23 16:11:13 -07:00
Jeremy Kemper
577034decb Ensure require and load are private - h/t apeiros 2010-04-24 10:38:41 -07:00
Jeremy Kemper
50cdb654ac Use call stack printer if available 2010-04-04 11:56:49 -07:00
Jeremy Kemper
4b07352420 ./tools/console to drop into irb with rails loaded 2010-03-28 18:18:25 -07:00
Jeremy Kemper
d73f2ff434 Rename tools/profile_requires -> tools/profile since we do both require and ruby-prof profiling. 2009-05-27 01:11:33 -05:00
Jeremy Kemper
886eeed52e Clean up tools/profile_requires a bit 2009-05-21 10:22:35 -07:00
Jeremy Kemper
4f291fa528 Simple examples for require profiling 2009-05-13 12:00:49 -07:00
Jeremy Kemper
e9c5750bd1 Preserve ordering 2009-05-13 11:43:03 -07:00
Jeremy Kemper
8ee0c598db Tool for profiling resource usage in each require call.
$ ruby -Iactiveresource/lib tools/profile_requires.rb active_resource
91.84 KB       3220 obj      4.0 ms  active_resource
83.86 KB       3108 obj      3.3 ms   active_support
69.32 KB       2682 obj      2.6 ms    active_support/vendor
33.98 KB        651 obj      0.6 ms     i18n
94.40 KB        315 obj      4.0 ms  44 KB RSS
2009-05-07 12:10:26 -07:00