Skip to content

Commit

Permalink
Merge remote-tracking branch 'proplib-template/main' into cross-platf…
Browse files Browse the repository at this point in the history
…orm-cli-driver
  • Loading branch information
aromanielloNTIA committed Nov 8, 2024
2 parents a11298e + d4146b5 commit eb05d93
Show file tree
Hide file tree
Showing 9 changed files with 17 additions and 13 deletions.
1 change: 0 additions & 1 deletion .github/workflows/ctest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ on:
# Define the matrix for different operating systems
jobs:
build-and-test:
if: ${{ github.repository != 'NTIA/proplib-template' }}
name: ${{ matrix.os }} / CMake ${{ matrix.cmakeVersion }}
runs-on: ${{ matrix.os }}
strategy:
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/doxygen.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@ on:
types: ["published"]
pull_request:
branches: ["main", "dev"]
push:
branches: ["main", "dev"]
workflow_dispatch:

# Allow only one concurrent deployment, skipping runs queued between the run in-progress and latest queued.
Expand All @@ -18,7 +20,6 @@ concurrency:

jobs:
build:
if: ${{ github.repository != 'NTIA/proplib-template' }}
runs-on: ubuntu-latest
steps:
- name: Checkout repository
Expand Down Expand Up @@ -55,7 +56,7 @@ jobs:
path: ./docs/html/

deploy:
if: ${{ (github.event_name == 'release') && (github.repository != 'NTIA/proplib-template') }}
if: ${{ github.event_name == 'release' }}
needs: build
permissions:
contents: read
Expand Down
1 change: 0 additions & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ permissions:

jobs:
create_release_artifacts:
if: github.repository != 'NTIA/proplib-template'
name: Create release artifacts
runs-on: ${{ matrix.os }}
strategy:
Expand Down
7 changes: 2 additions & 5 deletions app/src/CommaSeparatedIterator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,8 @@

#include "Driver.h"

#include <algorithm> // for transform
#include <cctype> // for std::tolower
#include <cstddef> // for std::ptrdiff_t, std::size_t
#include <fstream> // for std::istream
#include <iterator> // for std::input_iterator_tag
#include <cstddef> // for std::size_t
#include <istream> // for std::istream
#include <stdexcept> // for std::runtime_error
#include <string> // for std::getline, std::string

Expand Down
9 changes: 8 additions & 1 deletion app/src/Driver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,14 @@
*/
#include "Driver.h"

#include <algorithm> // for std::find
#include <algorithm> // for std::find
#include <fstream> // for std::ifstream, std::ofstream
#include <iomanip> // for std::setw
#include <ios> // for std::left
#include <iostream> // for std::cerr
#include <ostream> // for std::endl
#include <string> // for std::string
#include <vector> // for std::vector

/*******************************************************************************
* Main function of the driver executable
Expand Down
2 changes: 1 addition & 1 deletion app/src/DriverUtils.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -16,9 +16,9 @@
#include <algorithm> // for std::transform
#include <cctype> // for std::tolower
#include <ctime> // for localtime_{s,r}, std::{time, time_t, tm, strftime}
#include <fstream> // for std::ofstream
#include <iomanip> // for std::setfill, std::setw
#include <iostream> // for std::cerr, std::endl
#include <ostream> // for std::ostream
#include <string> // for std::stod, std::stoi, std::string

/******************************************************************************
Expand Down
2 changes: 1 addition & 1 deletion app/tests/TempTextFile.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@

#include <algorithm> // for std::remove
#include <fstream> // for std::ofstream
#include <iostream> // for std::cerr, std::cout, std::ios::trunc
#include <iostream> // for std::cerr, std::ios::trunc
#include <ostream> // for std::endl
#include <stdexcept> // for std::runtime_error
#include <string> // for std::string
Expand Down
2 changes: 2 additions & 0 deletions app/tests/TestDriver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
*/
#include "TestDriver.h"

#include <string> // for std::string

TEST_F(DriverTest, MissingOptionError1) {
// Test case: missing option between two provided flags
std::string cmd = executable + " -i -o out.txt";
Expand Down
1 change: 0 additions & 1 deletion app/tests/TestDriver.h
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
#include "Driver.h"
#include "TempTextFile.h"

#include <algorithm> // for std::replace
#include <cstdio> // for std::remove, std::perror
#include <cstdlib> // for std::system
#include <gtest/gtest.h> // GoogleTest
Expand Down

0 comments on commit eb05d93

Please sign in to comment.