<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/buildstream.git/tests/examples/first-project.py, branch bschubert/python38</title>
<subtitle>gitlab.com: buildstream/buildstream.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/'/>
<entry>
<title>Reformat code using Black</title>
<updated>2019-11-14T21:21:06+00:00</updated>
<author>
<name>Chandan Singh</name>
<email>csingh43@bloomberg.net</email>
</author>
<published>2019-11-11T17:07:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=122177153b14664a0e4fed85aa4f22b87cfabf56'/>
<id>122177153b14664a0e4fed85aa4f22b87cfabf56</id>
<content type='text'>
As discussed over the mailing list, reformat code using Black. This is a
one-off change to reformat all our codebase. Moving forward, we
shouldn't expect such blanket reformats. Rather, we expect each change
to already comply with the Black formatting style.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
As discussed over the mailing list, reformat code using Black. This is a
one-off change to reformat all our codebase. Moving forward, we
shouldn't expect such blanket reformats. Rather, we expect each change
to already comply with the Black formatting style.
</pre>
</div>
</content>
</entry>
<entry>
<title>test:utils/site: Consolidate IS_LINUX/WSL/Windows in a single site</title>
<updated>2019-06-06T14:34:00+00:00</updated>
<author>
<name>Benjamin Schubert</name>
<email>ben.c.schubert@gmail.com</email>
</author>
<published>2019-06-05T13:45:01+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=556f71c9a3ccdc4c524f261926a364379506066d'/>
<id>556f71c9a3ccdc4c524f261926a364379506066d</id>
<content type='text'>
We currently have two sites file containing redundant information.

- tests/testutils/site.py: Remove IS_{LINUX,WSL,WINDOWS}
- rest: Change imports to import those from
buildstream/tests/_utils/site.py
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We currently have two sites file containing redundant information.

- tests/testutils/site.py: Remove IS_{LINUX,WSL,WINDOWS}
- rest: Change imports to import those from
buildstream/tests/_utils/site.py
</pre>
</div>
</content>
</entry>
<entry>
<title>plugintestutils: Rename 'plugintestutils' package to 'testing'</title>
<updated>2019-04-16T13:25:40+00:00</updated>
<author>
<name>Phil Dawson</name>
<email>phil.dawson@codethink.co.uk</email>
</author>
<published>2019-04-15T10:02:33+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=97bbedbef15fc9edea5b04f66a30ec788da7f130'/>
<id>97bbedbef15fc9edea5b04f66a30ec788da7f130</id>
<content type='text'>
- Rename plugintestutils to testing.

- Don't run the tests from bst-plugins-template. This imports
  buildstream.plugintestutils so will have to be disabled to get
  through CI. This can be re nabled once bst-plugins-template has been
  patched.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
- Rename plugintestutils to testing.

- Don't run the tests from bst-plugins-template. This imports
  buildstream.plugintestutils so will have to be disabled to get
  through CI. This can be re nabled once bst-plugins-template has been
  patched.
</pre>
</div>
</content>
</entry>
<entry>
<title>tests: str(datafiles) instead of a longer thing</title>
<updated>2019-03-21T14:57:31+00:00</updated>
<author>
<name>Angelos Evripiotis</name>
<email>jevripiotis@bloomberg.net</email>
</author>
<published>2019-03-20T11:06:42+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=f2b15d86debd32241228f7b728016ed0057cd0d2'/>
<id>f2b15d86debd32241228f7b728016ed0057cd0d2</id>
<content type='text'>
Replace some popular copypasta.

This important-looking invocation:

    os.path.join(datafiles.dirname, datafiles.basename)

is equivalent to this shorter invocation:

    project = str(datafiles)

It seems like it's very popular copypasta, replace it with the shorter
one thus:

    # Use 'gsed' or 'sed' etc. as appropriate for your system.
    git config --global alias.sub '!f() { git grep --name-only --null "$1" | gxargs --null gsed --in-place --expression "s/$1/$2/g" ; }; f'
    git sub 'os.path.join(datafiles.dirname, datafiles.basename)' 'str(datafiles)'
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Replace some popular copypasta.

This important-looking invocation:

    os.path.join(datafiles.dirname, datafiles.basename)

is equivalent to this shorter invocation:

    project = str(datafiles)

It seems like it's very popular copypasta, replace it with the shorter
one thus:

    # Use 'gsed' or 'sed' etc. as appropriate for your system.
    git config --global alias.sub '!f() { git grep --name-only --null "$1" | gxargs --null gsed --in-place --expression "s/$1/$2/g" ; }; f'
    git sub 'os.path.join(datafiles.dirname, datafiles.basename)' 'str(datafiles)'
</pre>
</div>
</content>
</entry>
<entry>
<title>tests:lint: disable 'unused-import' checks on pytest fixtures</title>
<updated>2019-03-20T09:38:42+00:00</updated>
<author>
<name>Benjamin Schubert</name>
<email>ben.c.schubert@gmail.com</email>
</author>
<published>2019-03-19T18:36:38+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=da439e6658049bb79ca471f4eeac401e70c548f1'/>
<id>da439e6658049bb79ca471f4eeac401e70c548f1</id>
<content type='text'>
Pylint can't know that pytest's fixtures are used in a file and
therefore reports false positives. Silencing all those errors
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Pylint can't know that pytest's fixtures are used in a file and
therefore reports false positives. Silencing all those errors
</pre>
</div>
</content>
</entry>
<entry>
<title>tests:lint: silence redefined-outer-name in files using fixtures</title>
<updated>2019-03-20T09:38:42+00:00</updated>
<author>
<name>Benjamin Schubert</name>
<email>ben.c.schubert@gmail.com</email>
</author>
<published>2019-03-19T18:05:05+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=a90daa6a123d74a938a646e2fccaf4bfe6a5b950'/>
<id>a90daa6a123d74a938a646e2fccaf4bfe6a5b950</id>
<content type='text'>
Pylint doesn't play well with pytest fixtures, we therefore need to
silence this error.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Pylint doesn't play well with pytest fixtures, we therefore need to
silence this error.
</pre>
</div>
</content>
</entry>
<entry>
<title>tests: Remove unused parameters in functions</title>
<updated>2019-03-01T19:05:30+00:00</updated>
<author>
<name>Benjamin Schubert</name>
<email>ben.c.schubert@gmail.com</email>
</author>
<published>2019-02-28T19:35:18+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=402cc7af5464f34d29909148dfb0876e56f97124'/>
<id>402cc7af5464f34d29909148dfb0876e56f97124</id>
<content type='text'>
For parameters that are required as part of an API, prefix them
by "_" to make it clear they are unused
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
For parameters that are required as part of an API, prefix them
by "_" to make it clear they are unused
</pre>
</div>
</content>
</entry>
<entry>
<title>Expose basic api for testing external plugins.</title>
<updated>2019-02-08T14:27:54+00:00</updated>
<author>
<name>Phil Dawson</name>
<email>phil.dawson@codethink.co.uk</email>
</author>
<published>2019-01-16T14:03:41+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=b4d4c4f59c12c9b500f83d8cd4381418e1124f67'/>
<id>b4d4c4f59c12c9b500f83d8cd4381418e1124f67</id>
<content type='text'>
We want external plugins to be able to make use of the core testing utils.
This commit exposes the basic utilities which are currently in use in
bst-external plugins. If necessary, more utilities could be exposed in the
future.

Moves the following files from tests/testutils/ to
buildstream/plugintestingutils/:
o runcli.py
o integration.py

As part of this, this commit makes the following changes to runcli.py
and integration.py:
o runcli.py: Fix linting errors
o runcli.py: Add user facing documentation
o Integration.py: Add user facing documentation
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We want external plugins to be able to make use of the core testing utils.
This commit exposes the basic utilities which are currently in use in
bst-external plugins. If necessary, more utilities could be exposed in the
future.

Moves the following files from tests/testutils/ to
buildstream/plugintestingutils/:
o runcli.py
o integration.py

As part of this, this commit makes the following changes to runcli.py
and integration.py:
o runcli.py: Fix linting errors
o runcli.py: Add user facing documentation
o Integration.py: Add user facing documentation
</pre>
</div>
</content>
</entry>
<entry>
<title>Mark 'old' checkout command as obsolete</title>
<updated>2019-01-22T12:32:43+00:00</updated>
<author>
<name>James Ennis</name>
<email>james.ennis@codethink.com</email>
</author>
<published>2019-01-09T12:36:02+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=fbd159390e97a26f85bb7e8d6284b14bb77083ff'/>
<id>fbd159390e97a26f85bb7e8d6284b14bb77083ff</id>
<content type='text'>
This commit marks 'bst checkout' as a 'hidden' command. If used,
the user will be prompted to use the new 'bst artifact checkout'
command.

All tests which used 'bst checkout' have been modified to use
the new artifact sub-command.

This partially solves #822.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This commit marks 'bst checkout' as a 'hidden' command. If used,
the user will be prompted to use the new 'bst artifact checkout'
command.

All tests which used 'bst checkout' have been modified to use
the new artifact sub-command.

This partially solves #822.
</pre>
</div>
</content>
</entry>
<entry>
<title>doc: Adding part 1 of the getting started tutorial</title>
<updated>2018-06-16T20:14:04+00:00</updated>
<author>
<name>Tristan Van Berkom</name>
<email>tristan.vanberkom@codethink.co.uk</email>
</author>
<published>2018-06-16T17:20:40+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=1b88e2c6f0e3b060d7319e69d9ddf45bed98596f'/>
<id>1b88e2c6f0e3b060d7319e69d9ddf45bed98596f</id>
<content type='text'>
  o doc/Makefile: Added new directory to collect rst files from

  o doc/examples/first-project: Added the "first-project" example
    project.

  o doc/source/sessions/first-project-*.html: Added the generated
    snippets

  o doc/source/using_tutorial.rst: Added the new main tutorial page

  o doc/source/tutorial/first-project.rst: Added part 1 of the tutorial here

  o tests/examples/first-project.py: Added test for the example project

This is largely based on an example by Javier Jardón, which was
submitted at https://gitlab.com/BuildStream/buildstream/merge_requests/323

Fixes #103
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
  o doc/Makefile: Added new directory to collect rst files from

  o doc/examples/first-project: Added the "first-project" example
    project.

  o doc/source/sessions/first-project-*.html: Added the generated
    snippets

  o doc/source/using_tutorial.rst: Added the new main tutorial page

  o doc/source/tutorial/first-project.rst: Added part 1 of the tutorial here

  o tests/examples/first-project.py: Added test for the example project

This is largely based on an example by Javier Jardón, which was
submitted at https://gitlab.com/BuildStream/buildstream/merge_requests/323

Fixes #103
</pre>
</div>
</content>
</entry>
</feed>
